[GitHub] [incubator-celeborn] zhouyifan279 commented on pull request #1869: [CELEBORN-919][FOLLOWUP] Put map index args after partition index args in CelebornShuffleReader constructor

["zhouyifan279 (via GitHub)" <gi...@apache.org>]

zhouyifan279 commented on PR #1869:
URL: https://github.com/apache/incubator-celeborn/pull/1869#issuecomment-1700304431

   Note:
   1. In CelebornShuffleReader constructor, partition index args comes before map index args.
   2. In Celeborn SparkShuffleManager#getReader, map index args comes before partition index args.
   
   Due to gluten, we can not change the args order in  CelebornShuffleReader constructor.
   Due to Spark, we can not change the args order in SparkShuffleManager#getReader.


-- 
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-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org