wolfkill opened a new pull request, #18630:
URL: https://github.com/apache/pinot/pull/18630

   ## Summary
   - add a regression test for simple schema JSON that omits `fieldType` in 
metric, dimension, and time field specs
   - cover the compatibility case reported in #14017 without changing 
production code
   
   ## Why
   The reported schema now deserializes successfully on current `master`. This 
test keeps that compatibility covered so the older/simple schema format does 
not regress.
   
   Fixes #14017
   
   ## Tests
   - `JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home 
./mvnw -pl pinot-spi 
-Dtest=SchemaSerializationTest#testDeserializeSimpleSchemaWithoutFieldType test`
   - `JAVA_HOME=/opt/homebrew/opt/openjdk@21/libexec/openjdk.jdk/Contents/Home 
./mvnw -pl pinot-spi -Dtest=SchemaSerializationTest test`
   - `git diff --check`


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to