You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/01/24 11:59:02 UTC
[GitHub] [druid] clintropolis commented on issue #6974: sql support for
dynamic parameters
clintropolis commented on issue #6974: sql support for dynamic parameters
URL: https://github.com/apache/druid/pull/6974#issuecomment-578102368
> One note: given that parameters are specified in a list, could it be legal to leave out all ordinals and have them default to their 1-based list index if they are all missing?
@glasser I have modified the parameters in the http call to no longer have the ordinal property, in favor of the more user friendly approach you suggested of using list order. I have updated the PR description (and docs) to reflect this.
----------------------------------------------------------------
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org