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 ec70be7 [MINOR] Comments of PartitionBalanceAssignmentStrategy miss
byte units (#68)
ec70be7 is described below
commit ec70be79508f1a1d3a1e3a7becb548aa907e48a9
Author: jokercurry <[email protected]>
AuthorDate: Mon Jul 25 15:38:48 2022 +0800
[MINOR] Comments of PartitionBalanceAssignmentStrategy miss byte units (#68)
### What changes were proposed in this pull request?
Fixed the comment.
### Why are the changes needed?
Missing byte units.
### Does this PR introduce _any_ user-facing change?
No.
### How was this patch tested?
Without any test.
---
.../uniffle/coordinator/PartitionBalanceAssignmentStrategy.java | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
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 27d8e93..ba92477 100644
---
a/coordinator/src/main/java/org/apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java
+++
b/coordinator/src/main/java/org/apache/uniffle/coordinator/PartitionBalanceAssignmentStrategy.java
@@ -44,9 +44,9 @@ import org.apache.uniffle.common.PartitionRange;
* First round, we request one partition, then
* S1 (2G, 0) S2 (5G, 1) S3(1G, 0)
* Second round, we request one partition, then
- * S1 (2, 0) S2 (5G, 2) s3(1G, 0), we request one partition, then
+ * S1 (2G, 0) S2 (5G, 2) s3(1G, 0), we request one partition, then
* Third round, we request one partition, then
- * S1 (2, 1) S2 (5G, 2) s3(1G, 0)
+ * S1 (2G, 1) S2 (5G, 2) s3(1G, 0)
* ....
**/
