advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-524789505
@gatorsmile I believe only two occurrences of `new Configuration` are
invalid, and should be
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-524757373
@gatorsmile I did a quick search, most of the usage happens in XxxSuite
file, so it may be i
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-521167539
@HyukjinKwon tests are passed. Do you have any other concerns?
-
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-518921394
Gently ping @cloud-fan, @HyukjinKwon and @dongjoon-hyun.
---
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-516815646
> This patch fails due to an unknown error code, -9.
Still unrelated to this commit.
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-516696566
@HyukjinKwon The failure looks unrelated to this commit, could you trigger
the test again or
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-515843696
Gently ping @dongjoon-hyun.
Also cc @cloud-fan
-
advancedxy commented on issue #25002: [SPARK-28203][Core][Python] PythonRDD
should respect SparkContext's hadoop configuration
URL: https://github.com/apache/spark/pull/25002#issuecomment-514481630
@dongjoon-hyun sorry for the delay. I added a test case which should clearly
show the wrongl