shanielh commented on PR #10470: URL: https://github.com/apache/iceberg/pull/10470#issuecomment-2241581138
> @shanielh Sorry about missing to link the issue on #10433, that's totally my bad. As @singhpk234 said, that PR should address the issue you're seeing here. I'll be revisiting the PR this week. > > Although one interesting thing about this PR is that you're retrying SocketException, which I should also be retrying on my PR (currently retrying SocketTimeoutException for client side connection reset, but SocketException can occur and be retried for server side). No worries, I see this PR is still on draft, any chances to look @ my PR as I validated and it fixes an issue in real world situation where I read a long partition file and taking a long sleep between row groups, before this PR, when getting to process the second row group I get "SocketException". What is the expected time to merge #10433? -- 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
