JackieTien97 opened a new pull request, #17230: URL: https://github.com/apache/iotdb/pull/17230
This pull request updates the calculation logic for memory usage in the `AbstractTableScanOperator` and adds a new unit test to verify the changes. The main focus is on ensuring that the memory calculations correctly account for all sensor columns, including time and field columns, leading to more accurate resource estimation. **Memory calculation improvements:** * Updated the calculation of `maxReturnSize` in `AbstractTableScanOperator.java` to use `allSensors.size()` instead of `(1L + parameter.columnsIndexArray.length)`, ensuring all relevant columns are considered. * Modified the `calculateMaxPeekMemory()` method to return the maximum of `maxReturnSize` and the memory required for all sensors, improving the accuracy of memory usage estimation. **Testing enhancements:** * Added a new unit test `TableScanOperatorTest.java` to verify the updated memory calculation logic, ensuring correctness and preventing future regressions. -- 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]
