RussellSpitzer commented on PR #8523: URL: https://github.com/apache/iceberg/pull/8523#issuecomment-1758859862
My general gut on most apis is, Once they exist and have a defined behavior, we should only change that behavior or remove them if we have a good reason. I'm not sure this change protects any users and it could potentially harm existing users so I would vote against doing it. For future PR's I would also recommend adding tests for new behaviors. -- 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: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
