Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/1139#discussion_r171512422
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/handlers/DefaultSqlHandler.java
---
@@ -158,7 +162,9 @@ protected void logAndSetTextPlan(final String
description, final Prel prel, fina
protected void log(final String name, final PhysicalPlan plan, final
Logger logger) throws JsonProcessingException {
if (logger.isDebugEnabled()) {
- String planText =
plan.unparse(context.getLpPersistence().getMapper().writer());
+ PropertyFilter theFilter = new
SimpleBeanPropertyFilter.SerializeExceptFilter(Sets.newHashSet("password"));
--- End diff --
Please rename to `filter`.
---
