Caideyipi opened a new pull request, #18115:
URL: https://github.com/apache/iotdb/pull/18115

   ## Description
   
   Backport #18039 / 9dd1bb8 to `dev/1.3`.
   
   ### Metadata OOM diagnostics
   
   - Track matched, expanded, and generated result columns during query 
analysis.
   - Track schema-fetch deserialized column counts and schema tree memory 
reservation.
   - Enrich `MemoryNotEnoughException` with column counts, `SLIMIT`/`SOFFSET`, 
`ALIGN BY DEVICE` guidance, and memory details.
   - Add unit coverage for the enriched diagnostics.
   
   ### Branch adaptation
   
   `dev/1.3` does not have the newer query i18n source roots, so the diagnostic 
message templates are kept as local constants in `MPPQueryContext` for this 
branch.
   
   ### Verification
   
   - `mvn spotless:apply -pl iotdb-core/datanode`
   - `mvn "-Ddevelocity.off=true" -pl iotdb-core/datanode 
-Dtest=MPPQueryContextTest test` fails during datanode compilation before 
running the test, with unrelated compile errors around 
`PipePeriodicalLogReducer` and `FileUtils#createLink`.
   
   <hr>
   
   This PR has:
   - [x] been self-reviewed.
   - [ ] concurrent read
   - [ ] concurrent write
   - [ ] concurrent read and write
   - [ ] added documentation for new or modified features or behaviors.
   - [ ] added Javadocs for most classes and all non-trivial methods.
   - [ ] added or updated version, __license__, or notice information
   - [ ] added comments explaining the "why" and the intent of the code 
wherever would not be obvious for an unfamiliar reader.
   - [x] added unit tests or modified existing tests to cover new code paths, 
ensuring the threshold for code coverage.
   - [ ] added integration tests.
   - [ ] been tested in a test IoTDB cluster.
   
   <hr>
   
   ##### Key changed/added classes (or packages if there are too many classes) 
in this PR
   
   - `MPPQueryContext`
   - `ClusterSchemaTree`
   - `AnalyzeVisitor`
   - `ExpressionUtils`
   - `ClusterSchemaFetchExecutor`
   - `MPPQueryContextTest`


-- 
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]

Reply via email to