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 2019/05/30 18:59:01 UTC

[GitHub] [incubator-druid] gianm commented on issue #7798: Use smile format instead of json for internal Druid process communication.

gianm commented on issue #7798: Use smile format instead of json for internal Druid process communication.
URL: https://github.com/apache/incubator-druid/issues/7798#issuecomment-497445424
 
 
   This doesn't need to be done in 0.15.0, since HTTP supports negotiated response types: the client sends an Accept header with a preference-ranked list of content types that it supports, and the server picks one and responds with it, setting the Content-Type header to the one it chose. IMO, the best way to switch to smile is to adjust things at a deeper layer so _all_ Druid JSON endpoints support this kind of negotiation, and could return either JSON or Smile based on what the client supports and prefers.

----------------------------------------------------------------
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