Github user mattyb149 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2243#discussion_r157099185
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExecuteSQL.java
---
@@ -202,56 +205,63 @@ public void process(InputStream in) throws
IOException {
st.setQueryTimeout(queryTimeout); // timeout in seconds
logger.debug("Executing query {}", new Object[]{selectQuery});
- boolean results = st.execute(selectQuery);
+ boolean hasResults = st.execute(selectQuery);
+ boolean hasUpdateCount = st.getUpdateCount() != -1;
--- End diff --
I know you use MS SQL Server alot, does this call behave well? I saw
[this](https://stackoverflow.com/questions/21757191/jdbc-getupdatecount-is-returning-0-but-1-row-is-updated-in-sql-server)
and wondered if all is well without an executeUpdate() call.
---
