Skip to content

Conversation

@kgyrtkirk
Copy link
Member

No description provided.

}

private String generateLongErrorMessage() {
StringBuffer sb=new StringBuffer();
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a test, so not super important, but this should be a StringBuilder and please fix the checkstyle issue (no space between the =. Thanks.

@kgyrtkirk
Copy link
Member Author

I didn't wanted to get into trouble because of utf stuff; so I clip the message at 1K ; the purpose of this error message field is to given an idea what was the problem - and not to provide fully detailed backtrace/etc ; I think 1K might be already too much...in case some exception puts too much info into the "message" part of the exception....

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants