wangxianghu commented on a change in pull request #2994:
URL: https://github.com/apache/hudi/pull/2994#discussion_r642423048
##########
File path:
hudi-flink/src/main/java/org/apache/hudi/sink/partitioner/BucketAssignFunction.java
##########
@@ -271,34 +278,39 @@ private void loadRecords(String partitionPath) throws
Exception {
final int parallelism = getRuntimeContext().getNumberOfParallelSubtasks();
final int maxParallelism =
getRuntimeContext().getMaxNumberOfParallelSubtasks();
final int taskID = getRuntimeContext().getIndexOfThisSubtask();
+ final long currentTimeMillis = System.currentTimeMillis();
for (HoodieBaseFile baseFile : latestBaseFiles) {
+ String commitTime = baseFile.getCommitTime();
+ long commitTimeLong = instantDateFormat.parse(commitTime).getTime();
+ // file was expired, needn't to fetch hoodieKeys
+ if (ttlTime > 0 && (commitTimeLong + ttlTime) < currentTimeMillis) {
Review comment:
we can not add and compare them directly, they are not the same base
```
commitTimeLong -> yyyyMMddHHmmss
ttlTime -> number
currentTimeMillis -> number
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
