pdxcodemonkey opened a new pull request #838:
URL: https://github.com/apache/geode-native/pull/838


   From the bug description:  "The native client logger was rewritten several 
months ago to use, among other things, `std::put_time` when formatting log 
strings, specifically using the "%Z" formatting for timezone. The documentation 
for this formatting says "writes locale-dependent time zone name or 
abbreviation, or no characters if the time zone information is not available," 
and indeed it does. We just received a log file from a customer machine in APJ 
region, i.e. locale is set to something like Traditional Chinese. These strings 
are dumped into the log file as is, in some unknown MBCS encoding, and contain 
invalid utf-8 start codes, throwing off our Python-based parsing tool."
   
   The lower case `%z` format specifier just gives an offset from GMT, e.g. 
`-0700`, which might still be useful while also not dumping MBCS data into the 
logs, so that's what I went with.


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


Reply via email to