zuston commented on PR #396: URL: https://github.com/apache/incubator-uniffle/pull/396#issuecomment-1346120004
> Is there any read side related changes should be applied for this PR? There is no need to do any compatible change in read client, because the original logic has covered the different partition files due to the prefix of retry times. > And, if possible, please add an integration test for spark3 with concurrent writer enabled? It's OK. But the spark client test may be not accurate, because it's hard to control the concurrent write. And I think the integration test is enough and accurate. -- 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]
