Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2194#discussion_r145010425
--- Diff: nifi-nar-bundles/nifi-poi-bundle/nifi-poi-processors/pom.xml ---
@@ -77,6 +76,19 @@
</dependency>
<dependency>
<groupId>org.apache.nifi</groupId>
+ <artifactId>nifi-record-serialization-service-api</artifactId>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.nifi</groupId>
+ <artifactId>nifi-standard-record-utils</artifactId>
+ </dependency>
+ <dependency>
+ <groupId>org.apache.commons</groupId>
+ <artifactId>commons-csv</artifactId>
--- End diff --
I think by having a dependency to nifi-standard-record-utils, we don't have
to add commons-csv explicitly here. I'm fine with keeping it, but I prefer
having less dependencies.
---
