[GitHub] gianm commented on issue #6870: Enforce "dataSource" spelling via a Checkstyle regex

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

gianm commented on issue #6870: Enforce "dataSource" spelling via a Checkstyle regex
URL: https://github.com/apache/incubator-druid/issues/6870#issuecomment-454855211
 
 
   The unwritten style rule I have been following is that in JSON and in code (method/variable names) the proper spelling is "dataSource" or "DataSource". But in natural language documentation, the proper spelling is "datasource", because writing dataSource or DataSource looks strange in the middle of natural language text.
   
   If we were starting from scratch, I'd say call them "datasources" everywhere, but it'd be too disruptive to change all the JSON and code.

----------------------------------------------------------------
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: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org