danielcweeks commented on PR #6045: URL: https://github.com/apache/iceberg/pull/6045#issuecomment-1316034660
> @danielcweeks thx for your last round of review. I agree with you that UGI.currentUser is a better default owner, though I plan to make that change in a separate follow up PR. (See my reply above for more details) > > Also, I tried to rebase on master branch, but the change to `TableProperties` is still there. I'm fine separating UGI into a separate PR, but in that case we should hold off until the `1.1.0` release is finalized. I wouldn't want to introduce something that would like change behavior across the releases. -- 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
