kevinjqliu commented on PR #16507: URL: https://github.com/apache/iceberg/pull/16507#issuecomment-5957651193
Would appreciate a couple more eyes on the public API aspect of this change before merging. @rdblue @stevenzwu @danielcweeks would you have a chance to take a look? The main question for me is whether exposing the OAuth error type through `OAuth2Error` + the two existing HTTP exception subclasses is the right API shape, or whether we're adding too much OAuth-specific API surface. -- 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]
