[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558306997 Thank you for review, @databricks-david-lewis . Yes. I'm also suspecting that and checking now (if we

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558303060 Oh, interesting. We have a rule for that. Let me dig the reason how this escape from the checker. ```

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558301021 Thanks you, @srowen . Yes. I'll try to clean up the pom and dependency like this at this time.

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558292404 cc @databricks-david-lewis and @srowen