jcamachor commented on a change in pull request #1034:
URL: https://github.com/apache/hive/pull/1034#discussion_r431161927
##########
File path: ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java
##########
@@ -467,6 +467,8 @@
system.registerGenericUDAF("context_ngrams", new
GenericUDAFContextNGrams());
system.registerGenericUDAF("compute_stats", new GenericUDAFComputeStats());
+ system.registerGenericUDF("ndv_compute_bit_vector",
GenericUDFNDVComputeBitVector.class);
+ system.registerGenericUDAF("compute_bit_vector", new
GenericUDAFComputeBitVector());
Review comment:
I got feedback from Gopal about these names (I used something similar to
your suggestions first) and his reasoning to use these is that they should not
clash with any function that a user has previously added (same for the comment
you left above). That is why they have names that are understandable by us but
not necessarily straightforward. In any case, these are not user-facing
functions.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
