Uh oh!
There was an error while loading. Please reload this page.
[SPARK-35420][BUILD] Replace the usage of toStringHelper with ToStringBuilder - #32567
Closed
sarutak wants to merge 2 commits into
Closed
[SPARK-35420][BUILD] Replace the usage of toStringHelper with ToStringBuilder#32567sarutak wants to merge 2 commits into
sarutak wants to merge 2 commits into
Conversation
HyukjinKwon
approved these changes
May 17, 2021
HyukjinKwon
commented
May 17, 2021
Member
cc @srowen FYI |
SparkQA
commented
May 17, 2021
Test build #138600 has finished for PR 32567 at commit
|
SparkQA
commented
May 17, 2021
Kubernetes integration test unable to build dist. exiting with code: 1 |
dongjoon-hyun
commented
May 17, 2021
Member
The master branch compilation is recovered. Could you rebase to the master, @sarutak ? |
sarutak
commented
May 17, 2021
MemberAuthor
@dongjoon-hyun Thank you for letting me know! |
SparkQA
commented
May 17, 2021
Kubernetes integration test starting |
SparkQA
commented
May 17, 2021
Kubernetes integration test status failure |
SparkQA
commented
May 17, 2021
Test build #138604 has finished for PR 32567 at commit
|
sarutak
commented
May 17, 2021
MemberAuthor
Thanks all. Merging to |
sarutak added a commit
that referenced
this pull request
Jun 3, 2021
… which is incompatible with newer versions ### What changes were proposed in this pull request? This PR adds rules to `checkstyle.xml` and `scalastyle-config.xml` to avoid introducing `Objects.toStringHelper` a Guava's API which is no longer present in newer Guava. ### Why are the changes needed? SPARK-30272 (#26911) replaced `Objects.toStringHelper` which is an APIs Guava 14 provides with `commons.lang3` API because `Objects.toStringHelper` is no longer present in newer Guava. But toStringHelper was introduced into Spark again and replaced them in SPARK-35420 (#32567). I think it's better to have a style rule to avoid such repetition. SPARK-30272 replaced some APIs aside from `Objects.toStringHelper` but `Objects.toStringHelper` seems to affect Spark for now so I add rules only for it. ### Does this PR introduce _any_ user-facing change? No. ### How was this patch tested? I confirmed that `lint-java` and `lint-scala` detect the usage of `toStringHelper` and let the lint check fail. ``` $ dev/lint-java exec: curl --silent --show-error -L https://downloads.lightbend.com/scala/2.12.14/scala-2.12.14.tgz Using `mvn` from path: /opt/maven/3.6.3//bin/mvn Checkstyle checks failed at following occurrences: [ERROR] src/main/java/org/apache/spark/network/protocol/OneWayMessage.java:[78] (regexp) RegexpSinglelineJava: Avoid using Object.toStringHelper. Use ToStringBuilder instead. $ dev/lint-scala Scalastyle checks failed at following occurrences: [error] /home/kou/work/oss/spark/core/src/main/scala/org/apache/spark/rdd/RDD.scala:93:25: Avoid using Object.toStringHelper. Use ToStringBuilder instead. [error] Total time: 25 s, completed 2021/06/02 16:18:25 ``` Closes#32740 from sarutak/style-rule-for-guava. Authored-by: Kousuke Saruta <sarutak@oss.nttdata.com> Signed-off-by: Kousuke Saruta <sarutak@oss.nttdata.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changes were proposed in this pull request?
This PR replaces
toStringHelper, an API which breaks in Guava 27.Why are the changes needed?
SPARK-30272 (#26911) removed usages which breaks in Guava 27 but
toStringHelperis instroduced again.Does this PR introduce any user-facing change?
No.
How was this patch tested?
Build successfully finished with the following command.