JackieTien97 opened a new pull request, #16209: URL: https://github.com/apache/iotdb/pull/16209
This pull request introduces improvements to logging and memory management in the `AbstractWritableMemChunk` class, and makes minor adjustments to the `FragmentInstanceContext` class to simplify method access and improve thread safety. The most notable changes are the addition of logging for memory transfer retries and the removal of an unnecessary setter method. **Logging and Memory Management Enhancements:** * Added a logger to `AbstractWritableMemChunk` and enhanced the `maybeReleaseTvList` method to log a warning every 5 seconds when memory transfer to the query engine fails, helping with debugging and monitoring memory issues. [[1]](diffhunk://#diff-5665962b46995ddf0bb0f8d18d45a7d47f88562fcaccd62767412010fe0c6f63R36-R45) [[2]](diffhunk://#diff-5665962b46995ddf0bb0f8d18d45a7d47f88562fcaccd62767412010fe0c6f63R61-R72) **Thread Safety and Code Simplification:** * Removed the `setDataNodeQueryContext` setter from `FragmentInstanceContext`, making the context immutable after construction and reducing the risk of accidental modification. * Changed `releaseDataNodeQueryContext` in `FragmentInstanceContext` from a synchronized to a non-synchronized method, which may improve performance and reflects that synchronization is no longer needed in this context. -- 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]
