This is an automated email from the ASF dual-hosted git repository.
codope pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/hudi.git
The following commit(s) were added to refs/heads/master by this push:
new d28a0f2669 [MINOR] Fix Out of Bounds Exception for
DayBasedCompactionStrategy (#7360)
d28a0f2669 is described below
commit d28a0f26691f4b7c9ab243813ab7f80ca1b6d99c
Author: vamshigv <[email protected]>
AuthorDate: Mon Dec 12 00:59:59 2022 -0800
[MINOR] Fix Out of Bounds Exception for DayBasedCompactionStrategy (#7360)
---
.../hudi/table/action/compact/strategy/DayBasedCompactionStrategy.java | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git
a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/compact/strategy/DayBasedCompactionStrategy.java
b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/compact/strategy/DayBasedCompactionStrategy.java
index 94d74b50dc..424ce51d1e 100644
---
a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/compact/strategy/DayBasedCompactionStrategy.java
+++
b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/table/action/compact/strategy/DayBasedCompactionStrategy.java
@@ -78,7 +78,8 @@ public class DayBasedCompactionStrategy extends
CompactionStrategy {
public List<String> filterPartitionPaths(HoodieWriteConfig writeConfig,
List<String> allPartitionPaths) {
return allPartitionPaths.stream().map(partition -> partition.replace("/",
"-"))
.sorted(Comparator.reverseOrder()).map(partitionPath ->
partitionPath.replace("-", "/"))
- .collect(Collectors.toList()).subList(0,
writeConfig.getTargetPartitionsPerDayBasedCompaction());
+ .collect(Collectors.toList()).subList(0,
Math.min(allPartitionPaths.size(),
+ writeConfig.getTargetPartitionsPerDayBasedCompaction()));
}
/**
