This is an automated email from the ASF dual-hosted git repository.
yangjie01 pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/spark.git
The following commit(s) were added to refs/heads/master by this push:
new e8cccbd96cba [SPARK-54607][CORE] Remove unused method `toStringHelper`
from `AbstractFetchShuffleBlocks.java`
e8cccbd96cba is described below
commit e8cccbd96cba65eadd392551cd8deb5bc2f41168
Author: yangjie01 <[email protected]>
AuthorDate: Sat Dec 6 11:02:24 2025 +0800
[SPARK-54607][CORE] Remove unused method `toStringHelper` from
`AbstractFetchShuffleBlocks.java`
### What changes were proposed in this pull request?
This pr aims to remove the unused method `toStringHelper` from
`AbstractFetchShuffleBlocks.java` because it is no longer used after
https://github.com/apache/spark/pull/51572.
### Why are the changes needed?
Code cleanup.
### Does this PR introduce _any_ user-facing change?
No
### How was this patch tested?
- Pass Github Actions
### Was this patch authored or co-authored using generative AI tooling?
No
Closes #53341 from LuciferYang/remove-toStringHelper.
Authored-by: yangjie01 <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
---
.../network/shuffle/protocol/AbstractFetchShuffleBlocks.java | 11 -----------
1 file changed, 11 deletions(-)
diff --git
a/common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/protocol/AbstractFetchShuffleBlocks.java
b/common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/protocol/AbstractFetchShuffleBlocks.java
index 2bc57cc52f2c..6dc31b3c5958 100644
---
a/common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/protocol/AbstractFetchShuffleBlocks.java
+++
b/common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/protocol/AbstractFetchShuffleBlocks.java
@@ -21,8 +21,6 @@ import java.util.Objects;
import io.netty.buffer.ByteBuf;
-import org.apache.commons.lang3.builder.ToStringBuilder;
-import org.apache.commons.lang3.builder.ToStringStyle;
import org.apache.spark.network.protocol.Encoders;
/**
@@ -44,15 +42,6 @@ public abstract class AbstractFetchShuffleBlocks extends
BlockTransferMessage {
this.shuffleId = shuffleId;
}
- // checkstyle.off: RegexpSinglelineJava
- public ToStringBuilder toStringHelper() {
- return new ToStringBuilder(this, ToStringStyle.SHORT_PREFIX_STYLE)
- .append("appId", appId)
- .append("execId", execId)
- .append("shuffleId", shuffleId);
- }
- // checkstyle.on: RegexpSinglelineJava
-
/**
* Returns number of blocks in the request.
*/
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
