clintropolis commented on code in PR #16659: URL: https://github.com/apache/druid/pull/16659#discussion_r1683687688
########## docs/querying/scan-query.md: ########## @@ -228,3 +211,7 @@ Sample query context JSON object: "maxSegmentPartitionsOrderedInMemory": 100 } ``` + +## Legacy mode + +In older versions of Druid, the scan query supported a legacy mode designed for protocol compatibility with the former scan-query contrib extension from versions of Druid older than 0.11. This mode has been removed. Review Comment: i considered this, but then also thought it was kind of embarrassing that it took so long to remove it, so not really sure we needed to highlight that. Also I can't imagine that many people are using it, i think the only reason to do it is if you were originally using the contrib extension. The only reason I didn't remove it entirely from the docs was just in case people are searching for what it means in native query json stuff have something to reference. If you think there is some actionable purpose to specify which version it was removed in i can add it, but it doesn't seem too important to me. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
