exceptionfactory commented on PR #11704: URL: https://github.com/apache/nifi/pull/11704#issuecomment-5767713125
> @exceptionfactory Expended a bit on the ticket. > > > just adding this identifiesExternalResource() designation is not sufficient. > > Do you mean that regarding providing the `TEXT` type as well to the builder? I only added that as means to preserve the functionality of the `not.found` relationship. With only `FILE` added, the validator prevents the processor from running if the file is missing. So there is no intention of adding free text support to this processor. > > The goal is to have the FILE type generated into the manifest so third-party integrations can be aware of the actual type of the property. If `TEXT` is added, then the implementation needs to change to read the text content, but I agree that should not change given the nature of the Processor itself. There are other components that use just `FILE`, do you have an example of where that fails with only that addition? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
