singhpk234 commented on PR #5094: URL: https://github.com/apache/iceberg/pull/5094#issuecomment-1162859934
> I am a little bit confused. Shouldn't https://github.com/apache/iceberg/pull/5056 be reviewed and merged first? It was what I had in mind originally as well : 1. we would first merge https://github.com/apache/iceberg/pull/5056, which will make master have only 3.3 and not 3.2 2. then we would merge this pr adding back 3.2 and make it work with 3.3. since we started with this pr, I think post finalizing 3.3 changes will make this pr 3.3 only and move 3.2 to https://github.com/apache/iceberg/pull/5056. which @aokolnychyi suggested as well. -- 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]
