Add support for complex types to histogram Presto function (facebooki…

…ncubator#10297)

Summary:
Pull Request resolved: facebookincubator#10297

Add support for complex types and UNKNOWN to histogram aggregate function.

Reviewed By: amitkdutta

Differential Revision: D58973456

fbshipit-source-id: 1153bd629290951a78ddea8244ea3bf47a1e78b4

velox/exec/AddressableNonNullValueList.cpp

@@ -41,16 +41,22 @@ AddressableNonNullValueList::Entry AddressableNonNullValueList::append(
     const DecodedVector& decoded,
     vector_size_t index,
     HashStringAllocator* allocator) {
+  return append(*decoded.base(), decoded.index(index), allocator);
+}
+
+AddressableNonNullValueList::Entry AddressableNonNullValueList::append(
+    const BaseVector& vector,
+    vector_size_t index,
+    HashStringAllocator* allocator) {
   auto stream = initStream(allocator);
 
-  const auto hash = decoded.base()->hashValueAt(decoded.index(index));
+  const auto hash = vector.hashValueAt(index);
 
   const auto originalSize = stream.size();
 
   // Write value.
   exec::ContainerRowSerdeOptions options{};
-  exec::ContainerRowSerde::serialize(
-      *decoded.base(), decoded.index(index), stream, options);
+  exec::ContainerRowSerde::serialize(vector, index, stream, options);
   ++size_;
 
   auto startAndFinish = allocator->finishWrite(stream, 1024);

velox/exec/AddressableNonNullValueList.h

@@ -58,6 +58,12 @@ class AddressableNonNullValueList {
       vector_size_t index,
       HashStringAllocator* allocator);
 
+  /// Same as above, but takes BaseVector.
+  Entry append(
+      const BaseVector& vector,
+      vector_size_t index,
+      HashStringAllocator* allocator);
+
   /// Append a non-null serialized value to the end of the list.
   /// Returns position that can be used to access the value later.
   HashStringAllocator::Position appendSerialized(