[GitHub] [spark] gengliangwang commented on a change in pull request #29357: [SPARK-32539][INFRA] Disallow `FileSystem.get(Configuration conf)` in style check by default

2020-08-05 Thread GitBox
gengliangwang commented on a change in pull request #29357: URL: https://github.com/apache/spark/pull/29357#discussion_r466147119 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetInteroperabilitySuite.scala ## @@ -165,7 +165,7 @@

[GitHub] [spark] gengliangwang commented on a change in pull request #29357: [SPARK-32539][INFRA] Disallow `FileSystem.get(Configuration conf)` in style check by default

2020-08-05 Thread GitBox
gengliangwang commented on a change in pull request #29357: URL: https://github.com/apache/spark/pull/29357#discussion_r466104000 ## File path: sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetInteroperabilitySuite.scala ## @@ -165,7 +165,7 @@

[GitHub] [spark] gengliangwang commented on a change in pull request #29357: [SPARK-32539][INFRA] Disallow `FileSystem.get(Configuration conf)` in style check by default

2020-08-05 Thread GitBox
gengliangwang commented on a change in pull request #29357: URL: https://github.com/apache/spark/pull/29357#discussion_r465852807 ## File path: scalastyle-config.xml ## @@ -264,6 +264,19 @@ This file is divided into 3 sections: of Commons Lang 2 (package

[GitHub] [spark] gengliangwang commented on a change in pull request #29357: [SPARK-32539][INFRA] Disallow `FileSystem.get(Configuration conf)` in style check by default

2020-08-05 Thread GitBox
gengliangwang commented on a change in pull request #29357: URL: https://github.com/apache/spark/pull/29357#discussion_r465838231 ## File path: scalastyle-config.xml ## @@ -264,6 +264,19 @@ This file is divided into 3 sections: of Commons Lang 2 (package

[GitHub] [spark] gengliangwang commented on a change in pull request #29357: [SPARK-32539][INFRA] Disallow `FileSystem.get(Configuration conf)` in style check by default

2020-08-05 Thread GitBox
gengliangwang commented on a change in pull request #29357: URL: https://github.com/apache/spark/pull/29357#discussion_r465785861 ## File path: scalastyle-config.xml ## @@ -264,6 +264,19 @@ This file is divided into 3 sections: of Commons Lang 2 (package

[GitHub] [spark] gengliangwang commented on a change in pull request #29357: [SPARK-32539][INFRA] Disallow `FileSystem.get(Configuration conf)` in style check by default

2020-08-05 Thread GitBox
gengliangwang commented on a change in pull request #29357: URL: https://github.com/apache/spark/pull/29357#discussion_r465721589 ## File path: core/src/main/scala/org/apache/spark/deploy/security/HadoopFSDelegationTokenProvider.scala ## @@ -139,7 +139,9 @@ private[deploy]