refactor: Remove redundant tryMapToRange<StringView> declaration (facebookincubator#16385)

Summary:
VectorHasher has two tryMapToRange template specializations:
`tryMapToRange<bool>` and `tryMapToRange<StringView>`.  The
`tryMapToRange<StringView>` template specialization was declared
twice. This patch removes the redundant declaration.

No functional changes.

Pull Request resolved: facebookincubator#16385

Reviewed By: kKPulla

Differential Revision: D93285257

Pulled By: jainxrohit

fbshipit-source-id: ce314691513cae6bfef8c86736c547e24d376e04

Author: lingbin

velox/exec/VectorHasher.h

@@ -646,14 +646,6 @@ bool VectorHasher::makeValueIdsForRows<TypeKind::VARCHAR>(
 template <>
 void VectorHasher::analyzeValue(StringView value);
 
-template <>
-inline bool VectorHasher::tryMapToRange(
-    const StringView* /*values*/,
-    const SelectivityVector& /*rows*/,
-    uint64_t* /*result*/) {
-  return false;
-}
-
 template <>
 inline uint64_t VectorHasher::valueId(StringView value) {
   auto size = value.size();
@@ -748,10 +740,12 @@ inline bool VectorHasher::tryMapToRange(
 }
 
 template <>
-bool VectorHasher::tryMapToRange(
+inline bool VectorHasher::tryMapToRange(
     const StringView* /*values*/,
     const SelectivityVector& /*rows*/,
-    uint64_t* /*result*/);
+    uint64_t* /*result*/) {
+  return false;
+}
 
 template <>
 bool VectorHasher::makeValueIdsFlatNoNulls<bool>(