leventov commented on issue #6370: Introduce SegmentId class URL: https://github.com/apache/incubator-druid/pull/6370#issuecomment-428618440 @egor-ryashin > I think we shouldn't use heuristic methods like iterateAllPossibleParsings for all datasources, moreover if a production cluster doesn't contain ambigous datasource names and versions, then the operator should have an option to switch it off. `iterateAllPossibleParsings()` is used in endpoints accepting segment ids as strings. Performance penalty of `iterateAllPossibleParsings()` is insignificant compared to the cost of a HTTP call itself, and also such endpoints are not expected to be queried on coordinators or other nodes 10k times per second.
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
