[GitHub] [nifi] exceptionfactory commented on pull request #7194: NIFI-11167 - Add Excel Record Reader

["exceptionfactory (via GitHub)" <gi...@apache.org>]

exceptionfactory commented on PR #7194:
URL: https://github.com/apache/nifi/pull/7194#issuecomment-1584871036

   > @exceptionfactory In order to do this I do need many of the dependencies that are already in `nifi-schema-registry-service-api` as well as classes that are in that module e.g. (`org.apache.nifi.schema.inference.InferSchemaAccessStrategy` and `org.apache.nifi.schema.inference.SchemaInferenceUtil`). Do I just include that module as a dependency which whereby I would get the other dependencies as transient dependencies?
   
   It should be possible to add the various `service-api` dependencies as needed to a new `nifi-poi-services` module. That module should not depend on `nifi-record-serialization-services` itself, so it might be necessary to create similar implementations of one or two utility classes, but almost everything necessary should be available through the API modules. The `nifi-asn1-services` module is an example of a Record Reader outside of `nifi-record-serialization-services`.


-- 
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-unsubscribe@nifi.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org