tgravescs commented on issue #26611: [SPARK-29939][CORE] Add spark.shuffle.mapStatus.compression.codec conf URL: https://github.com/apache/spark/pull/26611#issuecomment-558205249 Yeah I definitely get that and we have left configs internal in the past, it just seemed here the only reason I saw was in case there is issue with Zstd and if we really think that could happen or has happened then maybe it shouldn't be internal. if there hasn't been then did we really need this config at all like was mentioned in original pr discussion. Anyway, I'm fine with it just wondering if more reasoning behind it. We can always open it up later.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
