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;