mattyb149 commented on pull request #5701: URL: https://github.com/apache/nifi/pull/5701#issuecomment-1042106843
@mi-volodin I didn't see the same behavior you describe, but did find a bug. I debugged the unit test and `normalizeKeyColumnNamesAndCheckForValues` doesn't quote the column names, just normalizes it by uppercasing it and removing underscores and such. Also the normalized column name it matches against isn't quoted, it should only be quoted when generating the SQL. However it was using `normalizedColumnName` as the column name in the `WHERE` clause when it should be using `desc.getColumnName()` like the `SET` clause does. That can cause issues, I will issue another commit to update the unit test to illustrate this, along with the fix to get the test to pass :) -- 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: issues-unsubscr...@nifi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
