Jackie-Jiang opened a new pull request, #18903:
URL: https://github.com/apache/pinot/pull/18903

   ## Summary
   Removes the unused `sortable` flag from `FieldSpec.DataType`:
   - Drops the `_sortable` field and the `canBeASortedColumn()` method, which 
has no callers anywhere in the codebase (dead code).
   - Drops the `sortable` field from `DataTypeProperties`, so it no longer 
appears in the `GET /schemas/fieldSpec` metadata response. No consumer 
(including the controller UI) reads this field.
   - Adds `@JsonIgnoreProperties(ignoreUnknown = true)` on `DataTypeProperties` 
so deserializers remain tolerant of payloads from older versions that still 
include `sortable`.
   - Simplifies the enum constructors accordingly.
   


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