You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by "dongjoon-hyun (via GitHub)" <gi...@apache.org> on 2023/09/23 23:16:29 UTC

[GitHub] [spark] dongjoon-hyun commented on pull request #41906: [SPARK-44345][CORE] Only log unknown shuffle map output as error when shuffle migration disabled

dongjoon-hyun commented on PR #41906:
URL: https://github.com/apache/spark/pull/41906#issuecomment-1732430297

   > @dongjoon-hyun Could you help take a look?
   
   Hi, @warrenzhu25 . I don't think https://github.com/apache/spark/pull/41906#discussion_r1278819867 is addressed properly. This PR is still not ready due to that. IIUC, he asked not to change it explicitly. In addition, that change is not consistent with the AS-IS PR title neither.


-- 
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: reviews-unsubscribe@spark.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org