ankitsultana commented on PR #10423: URL: https://github.com/apache/pinot/pull/10423#issuecomment-1469446573
Thanks.. this is good to see. One note: this is setting nullability at table-level. Most of our use-cases require nullability to be set at column-level. People want do things like IS NULL on some columns while at the same time they want to use sub-queries with semi-joins (`uuid IN (...)`), so they want `uuid` to be non-null. I think long-term we may want column level nullability, in which case we may need to think how backwards compatibility will be handled if we go with table level nullability right now. -- 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]
