Skip to content

Commit e8cccbd

Browse files
committed
[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 #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]>
1 parent 3d42605 commit e8cccbd

File tree

1 file changed

+0
-11
lines changed

1 file changed

+0
-11
lines changed

common/network-shuffle/src/main/java/org/apache/spark/network/shuffle/protocol/AbstractFetchShuffleBlocks.java

Lines changed: 0 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,6 @@
2121

2222
import io.netty.buffer.ByteBuf;
2323

24-
import org.apache.commons.lang3.builder.ToStringBuilder;
25-
import org.apache.commons.lang3.builder.ToStringStyle;
2624
import org.apache.spark.network.protocol.Encoders;
2725

2826
/**
@@ -44,15 +42,6 @@ protected AbstractFetchShuffleBlocks(
4442
this.shuffleId = shuffleId;
4543
}
4644

47-
// checkstyle.off: RegexpSinglelineJava
48-
public ToStringBuilder toStringHelper() {
49-
return new ToStringBuilder(this, ToStringStyle.SHORT_PREFIX_STYLE)
50-
.append("appId", appId)
51-
.append("execId", execId)
52-
.append("shuffleId", shuffleId);
53-
}
54-
// checkstyle.on: RegexpSinglelineJava
55-
5645
/**
5746
* Returns number of blocks in the request.
5847
*/

0 commit comments

Comments
 (0)