You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2021/10/04 18:13:54 UTC

[GitHub] [pinot] walterddr opened a new issue #7512: Clean up behavior of config parsing

walterddr opened a new issue #7512:
URL: https://github.com/apache/pinot/issues/7512


   some inconsistencies were discovered after https://github.com/apache/pinot/pull/7497
   
   For example, configuration values were parsed into Lists and then re-constructed back into String. However the return changes were not exactly desired (e.g. parsed as comma-separated string but put back together as Arrays Stringification). This prompts me to ask a question: what are the proper configuration parsing between String / Arrays and other non-primitive types. 
   
   Since our usages in pinot-spi are largely depend on apache common configuration package. we should 
   - add some test around our extended behavior that might or might not be different from apache common. and 
   - correct any inconsistent / non-desired behavior if necessary


-- 
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: commits-unsubscribe@pinot.apache.org

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