cryptoe commented on PR #17937: URL: https://github.com/apache/druid/pull/17937#issuecomment-2842096783
> > @FrankChen021 I am currently unsure about the usecase here. Adding different flavors of the same API has a bunch of issues. > > > > 1. Confuses users. > > 2. Currently we have DART/SQL endpoints which are similar. The exact payload works in both endpoints. If we support this, we need to add support for it on DART too. > > > > For all the above reasons I have my reservations against this PR. More 1 than 2 tbh. > > 1. Please see the linked issue for the background: [Enhancing Query Context Handling: Introducing SQL-Level SETTINGS, Raw SQL Support, System Table and Context Validation #17769](https://github.com/apache/druid/issues/17769) > > 2. DART and sql statement have been modified to support this Thanks for the context. -- 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]
