Reuse isPositiveInteger function

rui-mo · Apr 10, 2024 · 8a605a8 · 8a605a8
1 parent 89de5d3
commit 8a605a8
Show file tree

Hide file tree

Showing 6 changed files with 10 additions and 32 deletions.
diff --git a/velox/expression/FunctionSignature.cpp b/velox/expression/FunctionSignature.cpp
@@ -93,14 +93,6 @@ size_t findNextComma(const std::string& str, size_t start) {
 }
 
 namespace {
-/// Returns true only if 'str' contains digits.
-bool isPositiveInteger(const std::string& str) {
-  return !str.empty() &&
-      std::find_if(str.begin(), str.end(), [](unsigned char c) {
-        return !std::isdigit(c);
-      }) == str.end();
-}
-
 void validateBaseTypeAndCollectTypeParams(
     const std::unordered_map<std::string, SignatureVariable>& variables,
     const TypeSignature& arg,

diff --git a/velox/expression/SignatureBinder.cpp b/velox/expression/SignatureBinder.cpp
@@ -27,14 +27,6 @@ bool isAny(const TypeSignature& typeSignature) {
   return typeSignature.baseName() == "any";
 }
 
-/// Returns true only if 'str' contains digits.
-bool isPositiveInteger(const std::string& str) {
-  return !str.empty() &&
-      std::find_if(str.begin(), str.end(), [](unsigned char c) {
-        return !std::isdigit(c);
-      }) == str.end();
-}
-
 std::string buildCalculation(
     const std::string& variable,
     const std::string& calculation) {

diff --git a/velox/expression/tests/utils/ArgumentTypeFuzzer.cpp b/velox/expression/tests/utils/ArgumentTypeFuzzer.cpp
@@ -45,15 +45,6 @@ bool isDecimalBaseName(const std::string& typeName) {
 
   return normalized == "decimal";
 }
-
-/// Returns true only if 'str' contains digits.
-bool isPositiveInteger(const std::string& str) {
-  return !str.empty() &&
-      std::find_if(str.begin(), str.end(), [](unsigned char c) {
-        return !std::isdigit(c);
-      }) == str.end();
-}
-
 } // namespace
 
 void ArgumentTypeFuzzer::determineUnboundedIntegerVariables(

diff --git a/velox/type/fbhive/HiveTypeParser.cpp b/velox/type/fbhive/HiveTypeParser.cpp
@@ -27,13 +27,6 @@ using facebook::velox::Type;
 using facebook::velox::TypeKind;
 
 namespace {
-/// Returns true only if 'str' contains digits.
-bool isPositiveInteger(const std::string& str) {
-  return !str.empty() &&
-      std::find_if(str.begin(), str.end(), [](unsigned char c) {
-        return !std::isdigit(c);
-      }) == str.end();
-}
 
 bool isSupportedSpecialChar(char c) {
   static std::unordered_set<char> supported{'_', '$', '#'};

diff --git a/velox/type/parser/ParserUtil.cpp b/velox/type/parser/ParserUtil.cpp
@@ -52,4 +52,11 @@ std::pair<std::string, std::shared_ptr<const Type>> inferTypeWithSpaces(
       fieldName, typeFromString(allWords.substr(fieldName.size() + 1)));
 }
 
+
+bool isPositiveInteger(const std::string& str) {
+  return !str.empty() &&
+      std::find_if(str.begin(), str.end(), [](unsigned char c) {
+        return !std::isdigit(c);
+      }) == str.end();
+}
 } // namespace facebook::velox
diff --git a/velox/type/parser/ParserUtil.h b/velox/type/parser/ParserUtil.h
@@ -36,4 +36,7 @@ std::pair<std::string, TypePtr> inferTypeWithSpaces(
     std::vector<std::string>& words,
     bool cannotHaveFieldName = false);
 
+/// Returns true only if 'str' contains digits.
+bool isPositiveInteger(const std::string& str);
+
 } // namespace facebook::velox