chore: use errors.New to replace fmt.Errorf with no parameters will much better

cluster/store/meta_class.go

@@ -12,6 +12,7 @@
 package store
 
 import (
+ "errors"
  "fmt"
  "strings"
  "sync"
@@ -88,7 +89,7 @@ func (m *metaClass) ShardOwner(shard string) (string, uint64, error) {
  return "", 0, errShardNotFound
  }
  if len(x.BelongsToNodes) < 1 || x.BelongsToNodes[0] == "" {
- return "", 0, fmt.Errorf("owner node not found")
+ return "", 0, errors.New("owner node not found")
  }
  return x.BelongsToNodes[0], m.version(), nil
 }

cluster/transport/service.go

@@ -100,7 +100,7 @@ func (s *Service) Leader() string {
 func (s *Service) Open() error {
  s.log.Info("starting cloud rpc server ...", "address", s.address)
  if s.address == "" {
- return fmt.Errorf("address of rpc server cannot be empty")
+ return errors.New("address of rpc server cannot be empty")
  }
  ln, err := net.Listen("tcp", s.address)
  if err != nil {

test/benchmark_bm25/cmd/import.go

@@ -13,6 +13,7 @@ package cmd
 
 import (
  "context"
+ "errors"
  "fmt"
  "log"
  "strconv"
@@ -70,7 +71,7 @@ var importCmd = &cobra.Command{
  }
 
  if !ok {
- return fmt.Errorf("weaviate is not ready")
+ return errors.New("weaviate is not ready")
  }
 
  datasets, err := lib.ParseDatasetConfig(DatasetConfigPath)

test/benchmark_bm25/cmd/query.go

@@ -179,7 +179,7 @@ var queryCmd = &cobra.Command{
  }
 
  if !ok {
- return fmt.Errorf("weaviate is not ready")
+ return errors.New("weaviate is not ready")
  }
  log.Print("weaviate is ready")