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

   ## Description
   
   This PR unifies pipe pattern inclusion parsing on the dev/1.3 codebase.
   
   Changes:
   - Adds explicit `pattern.inclusion` and `path.inclusion` source/extractor 
keys.
   - Allows legacy `pattern`, explicit `pattern.inclusion`, legacy `path`, and 
explicit `path.inclusion` to be used together as inclusion patterns.
   - Removes the public parser restriction that rejected non-single 
union/exclusion patterns.
   - Includes inclusion/exclusion pattern/path keys in TSFile parser memory 
detection.
   - Adds focused parsing/pruning tests for mixed inclusion keys and path 
exclusions.
   
   ## Tests
   
   - `mvn spotless:apply -pl iotdb-core/node-commons,iotdb-core/datanode`
   - `mvn -pl iotdb-core/node-commons -Dtest=PipePatternParseTest test`
   - `mvn -pl iotdb-core/node-commons,iotdb-core/datanode 
-Dtest=PipePatternPruningTest -DfailIfNoTests=false 
-Dsurefire.failIfNoSpecifiedTests=false test`
   
   Note: `mvn -pl iotdb-core/datanode -Dtest=PipePatternPruningTest test` fails 
before tests when run without the modified `node-commons` in the same reactor 
because it resolves an older local/remote `node-commons` artifact; the reactor 
command above passes.


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