[Go & Cpp] Validate schema when opening a space (#69)

Signed-off-by: sunby <[email protected]>
milvus-io · Oct 8, 2023 · 8692767 · 8692767
1 parent c58990f
commit 8692767
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/cpp/src/storage/space.cpp b/cpp/src/storage/space.cpp
@@ -246,6 +246,8 @@ Result<std::unique_ptr<Space>> Space::Open(const std::string& uri, Options optio
     if (options.schema == nullptr) {
       return Status::InvalidArgument("schema should not be nullptr");
     }
+
+    RETURN_NOT_OK(options.schema->Validate());
     manifest = std::make_shared<Manifest>(options.schema);
     RETURN_NOT_OK(SafeSaveManifest(fs, path, manifest.get()));
   } else {

diff --git a/go/storage/space.go b/go/storage/space.go
@@ -100,6 +100,9 @@ func Open(uri string, opt options.Options) (*Space, error) {
 				log.Error("schema is nil")
 				return nil, errors.ErrSchemaIsNil
 			}
+			if err = opt.Schema.Validate(); err != nil {
+				return nil, err
+			}
 			m = manifest.NewManifest(opt.Schema)
 			m.SetVersion(0) //TODO: check if this is necessary
 			if err = rw.Write(m); err != nil {