dcapwell commented on code in PR #4024:
URL: https://github.com/apache/cassandra/pull/4024#discussion_r2021904227
##########
test/distributed/org/apache/cassandra/distributed/test/cql3/StatefulASTBase.java:
##########
@@ -504,7 +537,10 @@ private String humanReadable(Statement stmt, @Nullable
String annotate)
{
// With UTF-8 some chars can cause printing issues leading to
error messages that don't reproduce the original issue.
// To avoid this problem, always escape the CQL so nothing gets
lost
- String cql =
StringUtils.escapeControlChars(stmt.visit(debug).toCQL(CQLFormatter.None.instance));
+ CQLFormatter formatter = CQLFormatter.None.instance;
+ if (stmt.kind() == Statement.Kind.BATCH) // its really hard to
read these without this...
+ formatter = new CQLFormatter.PrettyPrint();
Review Comment:
i kinda want a new one, the sub elements are not formatted but the batch is
--
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: pr-unsubscr...@cassandra.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscr...@cassandra.apache.org
For additional commands, e-mail: pr-h...@cassandra.apache.org
