EndzeitBegins commented on PR #9782: URL: https://github.com/apache/nifi/pull/9782#issuecomment-2704716690
Thank you for looking into this @gresockj. Reverting the change #8390 certainly. However, is removing support for the property `REMOTE_POLL_BATCH_SIZE` from `ListSFTP` not essentially a breaking change, now that it present in previous versions? I'm aware of the discussions around a new release of NiFi on the mailing list soon. Wouldn't an easy, non-breaking fix (in terms of processor API) be to set the `maxResults` in [SFTPTransfer.java#L284](https://github.com/apache/nifi/blob/fb17c10b2fc2ebf78f96f7e9b105f2143c3b5235/nifi-extension-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/util/SFTPTransfer.java#L284) to a fixed value of `Integer.MAX_VALUE`. We could then look into a proper fix once the release is done. -- 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]
