[GitHub] [incubator-pinot] Jackie-Jiang commented on issue #6715: Add a flag in FieldSpec to indicate whether the _defaultNullValue is user-defined

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on issue #6715:
URL: https://github.com/apache/incubator-pinot/issues/6715#issuecomment-806231068


   You don't need another flag in the `FieldSpec`, instead you can add a method in `FieldSpec` which is json-ignored and return whether the `_stringDefaultNullValue` is `null` (whether the default is user-defined).
   I don't quite follow why you need to know whether the default is user-defined though. You can always leave null field as is and Pinot will fill it with default (either user-defined or not)


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org