westonpace commented on a change in pull request #9323: URL: https://github.com/apache/arrow/pull/9323#discussion_r568264814
########## File path: cpp/src/arrow/compute/api_vector.h ########## @@ -63,6 +63,25 @@ enum class SortOrder { Descending, }; +struct DictionaryEncodeOptions : public FunctionOptions { + /// Configure how null values will be encoded + enum NullEncodingBehavior { + /// the null value will be added to the dictionary with a proper index + ENCODE, + /// the null value will be masked in the indices array + MASK, + /// the null value will not be included in the dictionary + SKIP Review comment: I removed SKIP and added some comments similar to those in filter. ---------------------------------------------------------------- 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: us...@infra.apache.org