[GitHub] [druid] FrankChen021 commented on issue #10945: Scan query throws and error when using smile encoding for response

[GitBox <gi...@apache.org>]

FrankChen021 commented on issue #10945:
URL: https://github.com/apache/druid/issues/10945#issuecomment-791351416


   It's due to the implementation takes 'Accept' parameter which indicates smile encoding to decode the HTTP body which is actually a JSON.
   
   https://github.com/apache/druid/blob/16acd6686a337fc9a12e9aeda443fca973bdc625/server/src/main/java/org/apache/druid/server/QueryResource.java#L192
   
   https://github.com/apache/druid/blob/16acd6686a337fc9a12e9aeda443fca973bdc625/server/src/main/java/org/apache/druid/server/QueryResource.java#L196
   
   The input reader should alway decode the body according to 'Content-Type' header, while the output writer should encode the response by 'Accept' header. This problem might be introduced by #4033
   
   I'm wondering why you want to use smile ?


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org