[GitHub] spark issue #18842: [SPARK-21636] Several configurations which only are used...

2017-08-04 Thread liu-zhaokun
Github user liu-zhaokun commented on the issue: https://github.com/apache/spark/pull/18842 @jerryshao OK.Thanks. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and

[GitHub] spark issue #18842: [SPARK-21636] Several configurations which only are used...

2017-08-04 Thread jerryshao
Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/18842 I would suggest not to remove such configurations even if it is only used in UT, in case some users depend on them explicitly, abruptly removing them will break the compatibility. --- If your

[GitHub] spark issue #18842: [SPARK-21636] Several configurations which only are used...

2017-08-04 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/18842 No, these may be set by callers. It's not clear it's not on purpose. I would close this. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] spark issue #18842: [SPARK-21636] Several configurations which only are used...

2017-08-04 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/18842 Can one of the admins verify this patch? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this