You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uniffle.apache.org by ro...@apache.org on 2022/07/04 10:53:46 UTC
[incubator-uniffle] branch master updated: Covert PartitionAssignmentInfo to static inner class (#15)
This is an automated email from the ASF dual-hosted git repository.
roryqi pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-uniffle.git
The following commit(s) were added to refs/heads/master by this push:
new 3f3fce9 Covert PartitionAssignmentInfo to static inner class (#15)
3f3fce9 is described below
commit 3f3fce93480d7cc525afde01493ae9d0a07c118e
Author: Cheng Pan <pa...@gmail.com>
AuthorDate: Mon Jul 4 18:53:43 2022 +0800
Covert PartitionAssignmentInfo to static inner class (#15)
### What changes were proposed in this pull request?
Covert PartitionAssignmentInfo to static inner class.
### Why are the changes needed?
To avoid creating unnecessary outer class
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Existing UT.
---
.../apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/coordinator/src/main/java/org/apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java b/coordinator/src/main/java/org/apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java
index 3f91bbd..3dffb8b 100644
--- a/coordinator/src/main/java/org/apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java
+++ b/coordinator/src/main/java/org/apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java
@@ -135,7 +135,7 @@ public class PartitionBalanceAssignmentStrategy implements AssignmentStrategy {
return serverToPartitions;
}
- class PartitionAssignmentInfo {
+ static class PartitionAssignmentInfo {
PartitionAssignmentInfo() {
partitionNum = 0;