AMashenkov commented on a change in pull request #8821:
URL: https://github.com/apache/ignite/pull/8821#discussion_r581811412
##########
File path:
modules/indexing/src/main/java/org/apache/ignite/internal/processors/query/h2/sql/GridSqlStatement.java
##########
@@ -67,4 +73,19 @@ public void limit(GridSqlAst limit) {
public GridSqlAst limit() {
return limit;
}
+
+ /**
+ * @return Delimeter to use.
+ */
+ public static char delimeter() {
+ if (secureOutputToLog())
+ return SPACE_DELIM;
+
+ return DEFAULT_DELIM;
+ }
+
+ /** @return {@code True} if output is for log. */
+ public static boolean secureOutputToLog() {
+ return !GridSqlElement.INCLUDE_SENSITIVE &&
!GridSqlElement.INCLUDE_SENSITIVE_TL.get();
Review comment:
Let's replace this with QueryUtils.includeSensitive() and hide all logic
to utility class.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
