Skip to content

Conversation

@hellodengfei
Copy link

What changes were proposed in this pull request?

EventLog may contain mixed encodings such as custom exception message, but caused to replay failure

How was this patch tested?

manual test

@jiangxb1987
Copy link
Contributor

Can you also add test case for this?

@hellodengfei
Copy link
Author

Sorry, it's too late for UT.

@vanzin
Copy link
Contributor

vanzin commented May 11, 2018

Do you plan to address the feedback provided in jira about forcing UTF8 on the read and write paths?

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@HyukjinKwon
Copy link
Member

ping @hellodengfei

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants