chrisdennis opened a new pull request, #4455: URL: https://github.com/apache/calcite/pull/4455
To concrete things up this is what I'm suggesting as a new approach for CALCITE-7802. This implementation should (although nothing is tested) make sure that each property follows a simple priority model. Highest priority setting wins along the list: 1. System property 2. `saffron.properties` entry in `Thread.currentThread().getContextClassLoader()` (TCCL) accessible resource. 3. `saffron.properties` entry in `CalciteSystemPropery.class.getClassLoader()` accessible resource. For comparison the current implementation is either, with a TCCL set when Calcite is first loaded: 1. System property 2. `saffron.properties` entry in `saffron.properties` available via TCCL. with no TCCL set when Calcite is first loaded: 1. System property 2. `saffron.properties` entry in `saffron.properties` available via Calcite loader. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
