[GitHub] [spark] dongjoon-hyun commented on pull request #32697: [SPARK-31168][BUILD] Upgrade Scala to 2.12.14

2021-05-30 Thread GitBox
dongjoon-hyun commented on pull request #32697: URL: https://github.com/apache/spark/pull/32697#issuecomment-851237605 Sure, I agree with you that we need to remove it in Apache Spark 3.2.0 timeframe. Thanks, @iemejia . > My suggestion to remove silencer was to unblock the issue but thi

[GitHub] [spark] dongjoon-hyun commented on pull request #32697: [SPARK-31168][BUILD] Upgrade Scala to 2.12.14

2021-05-30 Thread GitBox
dongjoon-hyun commented on pull request #32697: URL: https://github.com/apache/spark/pull/32697#issuecomment-851075310 Thank you, @srowen , @iemejia , @viirya . - GitHub Action passed - Jenkins Scala/Java passed and currently SparkR test is running. I'll merge this PR for Apache Spa

[GitHub] [spark] dongjoon-hyun commented on pull request #32697: [SPARK-31168][BUILD] Upgrade Scala to 2.12.14

2021-05-30 Thread GitBox
dongjoon-hyun commented on pull request #32697: URL: https://github.com/apache/spark/pull/32697#issuecomment-851065745 Thank you, @viirya ! -- 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 sp

[GitHub] [spark] dongjoon-hyun commented on pull request #32697: [SPARK-31168][BUILD] Upgrade Scala to 2.12.14

2021-05-30 Thread GitBox
dongjoon-hyun commented on pull request #32697: URL: https://github.com/apache/spark/pull/32697#issuecomment-851059124 Now, genjavadoc is ready. - https://github.com/lightbend/genjavadoc/issues/282 -- This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [spark] dongjoon-hyun commented on pull request #32697: [SPARK-31168][BUILD] Upgrade Scala to 2.12.14

2021-05-29 Thread GitBox
dongjoon-hyun commented on pull request #32697: URL: https://github.com/apache/spark/pull/32697#issuecomment-850906811 Hi, @srowen and @iemejia . Yes, it does. As I wrote in the PR description, I made a PR to **silencer** and it's merged three hours ago and released. > silencer li