dyf6372 commented on issue #6709: add charset UTF-8 to log api
URL: https://github.com/apache/incubator-druid/pull/6709#issuecomment-444767512
 
 
   In [rfc4627](https://tools.ietf.org/html/rfc4627) section 3, the default 
encoding of JSON text is UTF-8.
   _JSON text SHALL be encoded in Unicode.  The default encoding is UTF-8._ 
   _Since the first two characters of a JSON text will always be ASCII 
characters [RFC0020], it is possible to determine whether an octet stream is 
UTF-8, UTF-16 (BE or LE), or UTF-32 (BE or LE) by looking at the pattern of 
nulls in the first four octets._
   Since it is possible to determine whether an octet stream is UTF-8, UTF-16, 
or UTF-32, I'm confused whether to set the charset to UTF-8 for 
MediaType.APPLICATION_JSON.  Maybe is same to MediaType.TEXT_PLAIN.
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to