vogievetsky commented on pull request #11908:
URL: https://github.com/apache/druid/pull/11908#issuecomment-977557250
Very excited for this change! I got 2 questions:
(1) is it possible to provide the signature as JSON and not as a string?
instead of `"signature": "{d0:STRING, a0:LONG}"`
provide it as `"signature": [{name: "d0", "type":"STRING"}, {"name":"a0",
"type":"LONG"}]`
the signature strings are really annoying to parse and these signatures are
really important (sometimes more so than then query)
(2) you included screenshots of the old format in the console query view,
but how does it work with the explain dialog? Does it need to be updated as
part of this PR or right afterwards?
List to be clear I am talking about this dialog:
As you can see the current dialog parses the old format thus relying on it.
I would LOVE nothing more than to switch to this new format (providing pt.1)
but does this PR break this dialog?
--
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]
