Merge pull request #1438 from kuzudb/change-copynpy-keyword

change copy from npy keyword
kuzudb · Apr 2, 2023 · f4a7914 · f4a7914
2 parents 380d8b8 + ec07a43
commit f4a7914
Show file tree

Hide file tree

Showing 12 changed files with 2,545 additions and 2,538 deletions.
diff --git a/dataset/npy-1d/copy.cypher b/dataset/npy-1d/copy.cypher
@@ -1 +1 @@
-copy npytable fromnpy ("dataset/npy-1d/one_dim_int64.npy",  "dataset/npy-1d/one_dim_int32.npy",  "dataset/npy-1d/one_dim_int16.npy",  "dataset/npy-1d/one_dim_double.npy", "dataset/npy-1d/one_dim_float.npy");
+npy copy npytable from ("dataset/npy-1d/one_dim_int64.npy",  "dataset/npy-1d/one_dim_int32.npy",  "dataset/npy-1d/one_dim_int16.npy",  "dataset/npy-1d/one_dim_double.npy", "dataset/npy-1d/one_dim_float.npy");
diff --git a/dataset/npy-20k/copy.cypher b/dataset/npy-20k/copy.cypher
@@ -1 +1 @@
-copy npytable fromNPY ("dataset/npy-20k/id_int64.npy", "dataset/npy-20k/two_dim_float.npy");
+npy copy npytable from ("dataset/npy-20k/id_int64.npy", "dataset/npy-20k/two_dim_float.npy");
diff --git a/dataset/npy-2d/copy.cypher b/dataset/npy-2d/copy.cypher
@@ -1 +1 @@
-copy npytable fromNPY ("dataset/npy-2d/id_int64.npy", "dataset/npy-2d/two_dim_int64.npy", "dataset/npy-2d/two_dim_int32.npy",  "dataset/npy-2d/two_dim_int16.npy",  "dataset/npy-2d/two_dim_double.npy", "dataset/npy-2d/two_dim_float.npy");
+npy copy npytable from ("dataset/npy-2d/id_int64.npy", "dataset/npy-2d/two_dim_int64.npy", "dataset/npy-2d/two_dim_int32.npy",  "dataset/npy-2d/two_dim_int16.npy",  "dataset/npy-2d/two_dim_double.npy", "dataset/npy-2d/two_dim_float.npy");
diff --git a/dataset/npy-3d/copy.cypher b/dataset/npy-3d/copy.cypher
@@ -1 +1 @@
-copy npytable FROMnpy ("dataset/npy-3d/id_int64.npy", "dataset/npy-3d/three_dim_int64.npy");
+npy copy npytable FROM ("dataset/npy-3d/id_int64.npy", "dataset/npy-3d/three_dim_int64.npy");
diff --git a/src/antlr4/Cypher.g4 b/src/antlr4/Cypher.g4
@@ -20,7 +20,7 @@ kU_CopyCSV
     : COPY SP oC_SchemaName SP FROM SP kU_FilePaths ( SP? '(' SP? kU_ParsingOptions SP? ')' )? ;
 
 kU_CopyNPY
-    : COPY SP oC_SchemaName SP FROMNPY ( SP '(' SP? StringLiteral ( SP? ',' SP? StringLiteral )* ')' ) ;
+    : NPY SP COPY SP oC_SchemaName SP FROM ( SP '(' SP? StringLiteral ( SP? ',' SP? StringLiteral )* ')' ) ;
 
 kU_FilePaths
     : '[' SP? StringLiteral ( SP? ',' SP? StringLiteral )* ']'
@@ -39,7 +39,7 @@ COPY : ( 'C' | 'c' ) ( 'O' | 'o' ) ( 'P' | 'p') ( 'Y' | 'y' ) ;
 
 FROM : ( 'F' | 'f' ) ( 'R' | 'r' ) ( 'O' | 'o' ) ( 'M' | 'm' );
 
-FROMNPY : ( 'F' | 'f' ) ( 'R' | 'r' ) ( 'O' | 'o' ) ( 'M' | 'm' ) ( 'N' | 'n' ) ( 'P' | 'p' ) ( 'Y' | 'y' ) ;
+NPY : ( 'N' | 'n' ) ( 'P' | 'p' ) ( 'Y' | 'y' ) ;
 
 kU_DDL
     : kU_CreateNode

diff --git a/src/binder/bind/bind_copy.cpp b/src/binder/bind/bind_copy.cpp
@@ -22,7 +22,7 @@ std::unique_ptr<BoundStatement> Binder::bindCopy(const Statement& statement) {
     std::unordered_map<common::property_id_t, std::string> propertyToNpyMap;
     if (expectedFileType == common::CopyDescription::FileType::UNKNOWN &&
         actualFileType == common::CopyDescription::FileType::NPY) {
-        throw BinderException("Please use COPY FROMNPY statement for copying npy files.");
+        throw BinderException("Please use NPY COPY statement for copying npy files.");
     }
     if (expectedFileType == common::CopyDescription::FileType::NPY &&
         actualFileType != expectedFileType) {

diff --git a/test/copy/copy_npy_test.cpp b/test/copy/copy_npy_test.cpp
@@ -246,7 +246,7 @@ TEST_F(CopyLargeNpyTest, CopyLargeNpyTest) {
 
 TEST_F(CopyNpyFaultTest, CopyNpyInsufficientNumberOfProperties) {
     conn->query("create node table npytable (id INT64,i64 INT64[12],PRIMARY KEY(id));");
-    validateCopyException("copy npytable fromnpy (\"" +
+    validateCopyException("npy copy npytable from (\"" +
                               TestHelper::appendKuzuRootPath("dataset/npy-3d/three_dim_int64.npy") +
                               "\");",
         "Binder exception: Number of npy files is not equal to number of properties in table "
@@ -256,7 +256,7 @@ TEST_F(CopyNpyFaultTest, CopyNpyInsufficientNumberOfProperties) {
 TEST_F(CopyNpyFaultTest, CopyNpyRedundantProperties) {
     conn->query("create node table npytable (id INT64,i32 INT32, PRIMARY KEY(id));");
     auto f32Path = TestHelper::appendKuzuRootPath("dataset/npy-20k/two_dim_float.npy");
-    validateCopyException("copy npytable fromnpy (\"" +
+    validateCopyException("npy copy npytable from (\"" +
                               TestHelper::appendKuzuRootPath("dataset/npy-20k/id_int64.npy") +
                               "\", \"" + f32Path + "\");",
         "The type of npy file " + f32Path + " does not match the type defined in table npytable.");
@@ -265,7 +265,7 @@ TEST_F(CopyNpyFaultTest, CopyNpyRedundantProperties) {
 TEST_F(CopyNpyFaultTest, CopyNpyVectorIntoScaler) {
     conn->query("create node table npytable (id INT64,f32 FLOAT, PRIMARY KEY(id));");
     auto f32Path = TestHelper::appendKuzuRootPath("dataset/npy-20k/two_dim_float.npy");
-    validateCopyException("copy npytable fromnpy (\"" +
+    validateCopyException("npy copy npytable from (\"" +
                               TestHelper::appendKuzuRootPath("dataset/npy-20k/id_int64.npy") +
                               "\", \"" + f32Path + "\");",
         "Cannot copy a vector property in npy file " + f32Path +
@@ -275,7 +275,7 @@ TEST_F(CopyNpyFaultTest, CopyNpyVectorIntoScaler) {
 TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedTypeOneDimensionTest) {
     conn->query("create node table npytable (id INT64,i32 INT32, PRIMARY KEY(id));");
     auto f32Path = TestHelper::appendKuzuRootPath("dataset/npy-1d/one_dim_float.npy");
-    validateCopyException("copy npytable fromnpy ( \"" +
+    validateCopyException("npy copy npytable from ( \"" +
                               TestHelper::appendKuzuRootPath("dataset/npy-1d/one_dim_int64.npy") +
                               "\", \"" + f32Path + "\");",
         "The type of npy file " + f32Path + " does not match the type defined in table npytable.");
@@ -284,7 +284,7 @@ TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedTypeOneDimensionTest) {
 TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedTypeTwoDimensionTest) {
     conn->query("create node table npytable (id INT64,i32 INT32[10], PRIMARY KEY(id));");
     auto f32Path = TestHelper::appendKuzuRootPath("dataset/npy-20k/two_dim_float.npy");
-    validateCopyException("copy npytable fromnpy (\"" +
+    validateCopyException("npy copy npytable from (\"" +
                               TestHelper::appendKuzuRootPath("dataset/npy-20k/id_int64.npy") +
                               "\", \"" + f32Path + "\");",
         "The type of npy file " + f32Path + " does not match the type defined in table npytable.");
@@ -293,7 +293,7 @@ TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedTypeTwoDimensionTest) {
 TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedDimensionTest) {
     conn->query("create node table npytable (id INT64,f32 FLOAT[12],PRIMARY KEY(id));");
     auto twoDimFloatNpyPath = TestHelper::appendKuzuRootPath("dataset/npy-20k/two_dim_float.npy");
-    validateCopyException("copy npytable fromnpy (\"" +
+    validateCopyException("npy copy npytable from (\"" +
                               TestHelper::appendKuzuRootPath("dataset/npy-20k/id_int64.npy") +
                               "\", \"" + twoDimFloatNpyPath + "\");",
         "The shape of " + twoDimFloatNpyPath +
@@ -303,7 +303,7 @@ TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedDimensionTest) {
 TEST_F(CopyNpyFaultTest, CopyNpyWithMismatchedLengthTest) {
     conn->query("create node table npytable (id INT64,i64 INT64[12],PRIMARY KEY(id));");
     validateCopyException(
-        "copy npytable fromnpy (\"" +
+        "npy copy npytable from (\"" +
             TestHelper::appendKuzuRootPath("dataset/npy-20k/id_int64.npy") + "\", \"" +
             TestHelper::appendKuzuRootPath("dataset/npy-3d/three_dim_int64.npy") + "\");",
         "Number of rows in npy files is not equal to each other.");