cloud-fan commented on issue #24025: [SPARK-27106][SQL] merge CaseInsensitiveStringMap and DataSourceOptions URL: https://github.com/apache/spark/pull/24025#issuecomment-471142792 > So that the keys PATH_KEY/CHECK_FILES_EXIST_KEY/etc and their related methods can be also reserved. One goal is to remove these pre-defined option keys, as the options should just be a general string-to-string map. I don't think it's a good idea to keep both `CaseInsensitiveStringMap` and `DataSourceOptions` just for keeping code diff small. It will hurt long term maintainability.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
