FrankChen021 commented on pull request #10363: URL: https://github.com/apache/druid/pull/10363#issuecomment-690837225
@jihoonson @suneet-s I come up with an idea that requires no change to the existing configuration. 1. bind configuration path `druid.segmentCache` to `StorageLocationSelectorStrategy` directly 2. change the value of property in JsonTypeInfo annotation of `StorageLocationSelectorStrategy` from `type` to `locationSelectorStrategy` The disadvantage is that all properties extended by implementations of `StorageLocationSelectorStrategy` in the future will all be put under `druid.segmentCache`, which means these properties are mixed up with properties of `SegmentLoaderConfig`. This approach might cause some confusion. What do u think ? ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
