deemoliu commented on PR #13572: URL: https://github.com/apache/pinot/pull/13572#issuecomment-2218725864
> IIRC, currently Pinot doesn't support ingesting 0-length MV value, and that is the reason why we are not using empty list as default thanks @Jackie-Jiang for the review. i am aware that for multi-value the default null value is same as the default null value of a single-value columns, and the cardinality of the empty list for multi-value column is 1 instead of 0. there are two solution we provided to customer 1. emit another value instead of empty list to differentiate with null value, to unblock the user queries 2. we make some change in the avro decoder to differentiate empty List and null values, since from upstream they are different Avro values. To avoid breaking the backward compatibility, we added a config instead of changing the default behavior, is there any concerns for adding this config? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
