Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-06-14 Thread via GitHub
pnowojski closed pull request #24211: [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator URL: https://github.com/apache/flink/pull/24211 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-06-14 Thread via GitHub
pnowojski commented on PR #24211: URL: https://github.com/apache/flink/pull/24211#issuecomment-2168078012 I've prepared an improved version of this PR: https://github.com/apache/flink/pull/24941 Closing this in favour the newer one. -- This is an automated message from the Apache G

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-23 Thread via GitHub
pnowojski commented on PR #24211: URL: https://github.com/apache/flink/pull/24211#issuecomment-1960948515 > Do we know if watermarkInterval == 0 is a valid case where we are sure we need to periodically emit WatermarkStatus? My interpretation when first reading this code was that a zero val

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-22 Thread via GitHub
dchristle commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1499603545 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-22 Thread via GitHub
dchristle commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1499603545 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-20 Thread via GitHub
1996fanrui commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1495910626 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-20 Thread via GitHub
pnowojski commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1495762047 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-20 Thread via GitHub
pnowojski commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1495762047 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-19 Thread via GitHub
1996fanrui commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1495341757 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-02-19 Thread via GitHub
pnowojski commented on code in PR #24211: URL: https://github.com/apache/flink/pull/24211#discussion_r1494192141 ## flink-table/flink-table-runtime/src/main/java/org/apache/flink/table/runtime/operators/wmassigners/WatermarkAssignerOperator.java: ## @@ -100,11 +100,14 @@ public

Re: [PR] [FLINK-34252][table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-01-28 Thread via GitHub
flinkbot commented on PR #24211: URL: https://github.com/apache/flink/pull/24211#issuecomment-1913851473 ## CI report: * 3b898a0c74a5dcb957ab7ab30b79dd82d01e8c0e UNKNOWN Bot commands The @flinkbot bot supports the following commands: - `@flinkbot run azure`

[PR] [FLINK-34252] [table] Fix lastRecordTime tracking in WatermarkAssignerOperator [flink]

2024-01-28 Thread via GitHub
dchristle opened a new pull request, #24211: URL: https://github.com/apache/flink/pull/24211 ## What is the purpose of the change In the current implementation, the lastRecordTime variable, which tracks the time of the last received data element, is updated only when the Wate