edit error message

vesoft-inc · Oct 28, 2024 · 00ba5c9 · 00ba5c9
1 parent 96d43e4
commit 00ba5c9
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 7 deletions.
diff --git a/src/graph/executor/mutate/DeleteExecutor.cpp b/src/graph/executor/mutate/DeleteExecutor.cpp
@@ -45,7 +45,8 @@ folly::Future<Status> DeleteVerticesExecutor::deleteVertices() {
         continue;
       }
       if (!SchemaUtil::isValidVid(val, *spaceInfo.spaceDesc.vid_type_ref())) {
-        auto errorMsg = fmt::format("Wrong vid type `{}`, value `{}`", val.type(), val.toString());
+        auto errorMsg = fmt::format(
+            "Wrong vid type `{}`, value `{}`", Value::toString(val.type()), val.toString());
         return Status::Error(errorMsg);
       }
       vertices.emplace_back(std::move(val));
@@ -102,7 +103,8 @@ folly::Future<Status> DeleteTagsExecutor::deleteTags() {
       continue;
     }
     if (!SchemaUtil::isValidVid(val, *spaceInfo.spaceDesc.vid_type_ref())) {
-      auto errorMsg = fmt::format("Wrong vid type `{}`, value `{}`", val.type(), val.toString());
+      auto errorMsg = fmt::format(
+          "Wrong vid type `{}`, value `{}`", Value::toString(val.type()), val.toString());
       return Status::Error(errorMsg);
     }
     delTag.id_ref() = val;

diff --git a/src/graph/executor/mutate/UpdateExecutor.cpp b/src/graph/executor/mutate/UpdateExecutor.cpp
@@ -58,7 +58,8 @@ folly::Future<Status> UpdateVertexExecutor::execute() {
         continue;
       }
       if (!SchemaUtil::isValidVid(val, *spaceInfo.spaceDesc.vid_type_ref())) {
-        auto errorMsg = fmt::format("Wrong vid type `{}`, value `{}`", val.type(), val.toString());
+        auto errorMsg = fmt::format(
+            "Wrong vid type `{}`, value `{}`", Value::toString(val.type()), val.toString());
         return Status::Error(errorMsg);
       }
       vertices.emplace_back(std::move(val));

diff --git a/src/graph/validator/MutateValidator.cpp b/src/graph/validator/MutateValidator.cpp
@@ -320,7 +320,7 @@ Status DeleteVerticesValidator::validateImpl() {
     if (type.value() != vidType_) {
       auto errorMsg = fmt::format("The vid `{}` should be type of `{}`, but was `{}`",
                                   vidRef_->toString(),
-                                  vidType_,
+                                  Value::toString(vidType_),
                                   type.value());
       return Status::SemanticError(errorMsg);
     }
@@ -463,7 +463,7 @@ Status DeleteTagsValidator::validateImpl() {
     if (type.value() != vidType_) {
       auto errorMsg = fmt::format("The vid `{}` should be type of `{}`, but was `{}`",
                                   vidRef_->toString(),
-                                  vidType_,
+                                  Value::toString(vidType_),
                                   type.value());
       return Status::SemanticError(errorMsg);
     }
@@ -675,7 +675,8 @@ Status UpdateValidator::getCondition() {
     auto type = typeStatus.value();
     if (type != Value::Type::BOOL && type != Value::Type::NULLVALUE &&
         type != Value::Type::__EMPTY__) {
-      auto errorMsg = fmt::format("`{}`, expected Boolean, but was `{}`", filter->toString(), type);
+      auto errorMsg = fmt::format(
+          "`{}`, expected Boolean, but was `{}`", filter->toString(), Value::toString(type));
       return Status::SemanticError(errorMsg);
     }
     condition_ = filter->encode();
@@ -853,7 +854,7 @@ Status UpdateVertexValidator::validateImpl() {
     if (type.value() != vidType_) {
       auto errorMsg = fmt::format("The vid `{}` should be type of `{}`, but was `{}`",
                                   vidRef_->toString(),
-                                  vidType_,
+                                  Value::toString(vidType_),
                                   type.value());
       return Status::SemanticError(errorMsg);
     }