You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/04/09 13:04:57 UTC

[GitHub] [spark] attilapiros commented on issue #24322: [SPARK-27412][Shuffle][WIP]Add a new shuffle manager: PmemShuffleManager

attilapiros commented on issue #24322: [SPARK-27412][Shuffle][WIP]Add a new shuffle manager: PmemShuffleManager
URL: https://github.com/apache/spark/pull/24322#issuecomment-481242722
 
 
   Yes, I have seen the design doc. 
   
   NIT: In the PR title you should use [Core] instead of [Shuffle]. 
   That is a category which used on the Spark PR UI: https://spark-prs.appspot.com/. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

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