timurturbil opened a new pull request, #8259: URL: https://github.com/apache/gravitino/pull/8259
### What changes were proposed in this pull request? Initialize `lastRecordDropEventTime` in `AsyncQueueListener` to avoid a potential `NullPointerException` when logging drop events. This ensures the timestamp is always non-null before being accessed. ### Why are the changes needed? `AsyncQueueListener` can throw a `NullPointerException` if the queue is full and drop events are logged before `lastRecordDropEventTime` is initialized. By setting a default value (`Instant.EPOCH`), we guarantee safe access and correct logging behavior. Fix: #8206 ### Does this PR introduce _any_ user-facing change? No. This change only affects the internal implementation of event logging. No user-facing APIs, configurations, or behaviors are modified. ### How was this patch tested? This patch was tested with the unit test provided by the issue author (`TestAsyncQueueListener.testDropEventLoggingDoesNotThrow`), which verifies that dropping events no longer throws an exception but the test is not included in the commit. -- 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]
