rdblue commented on pull request #2496: URL: https://github.com/apache/iceberg/pull/2496#issuecomment-1048416040
@wmoustafa, I think I mostly agree with you. That's probably how it should work. The only problem I have is point 3. Iceberg should not allow changing default values because it breaks the expected SQL behavior. That said, I would allow changing default values _only_ behind the `allowIncompatibleChanges` flag on the schema update operation. -- 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]
