mattyb149 commented on a change in pull request #4350:
URL: https://github.com/apache/nifi/pull/4350#discussion_r443679370
##########
File path:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/PutDatabaseRecord.java
##########
@@ -156,7 +162,7 @@
+ "FlowFile. The 'Use statement.type Attribute' option is
the only one that allows the 'SQL' statement type. If 'SQL' is specified, the
value of the field specified by the "
+ "'Field Containing SQL' property is expected to be a
valid SQL statement on the target database, and will be executed as-is.")
Review comment:
We should add some doc here explaining that some Statement Type values
may not be supported depending on the value of the Database Type property, and
can even include an example ("For example, UPSERT statements are not supported
by Oracle" or something like that)
----------------------------------------------------------------
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:
us...@infra.apache.org
