sunchao commented on PR #3692: URL: https://github.com/apache/celeborn/pull/3692#issuecomment-4536388366
@RexXiong Thanks for reviewing. The requested rollback switch is added in d44fa479d as `celeborn.client.spark.batch.openStream.parallelClientCreation.enabled`, defaulting to `true`; setting it to `false` restores the pre-change serial client-creation and request-building path. For the retry and warning-count point, I kept the existing semantics intentionally: before this PR, if `createClient` failed for a location, `workerRequestMap` remained empty and a later location on the same `hostPort` attempted `createClient` again, with failure handling per failed attempt. Reducing this to one attempt would be a separate behavioral change rather than preserving the existing reader behavior. -- 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]
