[GitHub] jfim opened a new pull request #3505: Handle invalid durations in the duration DSL

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

jfim opened a new pull request #3505: Handle invalid durations in the duration DSL
URL: https://github.com/apache/incubator-pinot/pull/3505
 
 
   Handle invalid durations in the duration DSL by removing invalid values
   when loading (eg. "null DAYS" or "null null"). These values come from
   invalid old-style configurations that got migrated to the new config
   format.
   
   This does not fully round-trip. For example, an old config with timeUnit
   of DAYS and timeValue of null will lose the DAYS qualifier, although
   this configuration was invalid (and ignored) in the first place.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

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