Github user MikeThomsen commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2625#discussion_r180728070
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryDatabaseTable.java
---
@@ -303,9 +303,8 @@ public void onTrigger(final ProcessContext context,
final ProcessSessionFactory
final Integer queryTimeout =
context.getProperty(QUERY_TIMEOUT).evaluateAttributeExpressions().asTimePeriod(TimeUnit.SECONDS).intValue();
st.setQueryTimeout(queryTimeout); // timeout in seconds
- try {
- logger.debug("Executing query {}", new
Object[]{selectQuery});
- final ResultSet resultSet = st.executeQuery(selectQuery);
+ logger.debug("Executing query {}", new Object[]{selectQuery});
--- End diff --
Should be wrapped with `if (logger.isDebugEnabled()) {}`
---
