Dwrite opened a new pull request, #5206: URL: https://github.com/apache/calcite/pull/5206
The behavior of SESSION table functions is unspecified for NULL timestamps Jira Link [CALCITE-7691](https://issues.apache.org/jira/browse/CALCITE-7691) Changes Proposed The behavior of the SESSION table function was undefined when the timestamp column contained NULL values: SessionizationEnumerator.initialize() called requireNonNull on the watermark column, so any row with a NULL timestamp caused a NullPointerException at runtime instead of a well-defined result. This is a one-line change that makes rows with a NULL timestamp be silently dropped from the session computation, rather than throwing. This matches the behavior of most SQL aggregate functions, which ignore NULL inputs, and avoids changing the (currently NOT NULL) return type of the window_start and window_end columns. A test case has been added to stream.iq covering a SESSION query whose input contains a mix of NULL and non-NULL timestamps. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
