You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@nifi.apache.org by "pvillard31 (via GitHub)" <gi...@apache.org> on 2025/08/03 18:05:05 UTC

[PR] NIFI-8269 - Add support for schema inference in ForkRecord processor when extracting array records [nifi]

pvillard31 opened a new pull request, #10167:
URL: https://github.com/apache/nifi/pull/10167

   # Summary
   
   NIFI-8269 - Add support for schema inference in ForkRecord processor when extracting array records
   
   > At the moment the ForkRecord processor will require the writer to be configured with a schema corresponding to what will be output. We could make things more clever to inherit the schema from the reader and change it based on the operation being performed.
   
   # Tracking
   
   Please complete the following tracking steps prior to pull request creation.
   
   ### Issue Tracking
   
   - [ ] [Apache NiFi Jira](https://issues.apache.org/jira/browse/NIFI) issue created
   
   ### Pull Request Tracking
   
   - [ ] Pull Request title starts with Apache NiFi Jira issue number, such as `NIFI-00000`
   - [ ] Pull Request commit message starts with Apache NiFi Jira issue number, as such `NIFI-00000`
   
   ### Pull Request Formatting
   
   - [ ] Pull Request based on current revision of the `main` branch
   - [ ] Pull Request refers to a feature branch with one commit containing changes
   
   # Verification
   
   Please indicate the verification steps performed prior to pull request creation.
   
   ### Build
   
   - [ ] Build completed using `mvn clean install -P contrib-check`
     - [ ] JDK 21
   
   ### Licensing
   
   - [ ] New dependencies are compatible with the [Apache License 2.0](https://apache.org/licenses/LICENSE-2.0) according to the [License Policy](https://www.apache.org/legal/resolved.html)
   - [ ] New dependencies are documented in applicable `LICENSE` and `NOTICE` files
   
   ### Documentation
   
   - [ ] Documentation formatting appears as expected in rendered files
   


-- 
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

Re: [PR] NIFI-8269 - Add support for schema inference in ForkRecord processor when extracting array records [nifi]

Posted by "pvillard31 (via GitHub)" <gi...@apache.org>.
pvillard31 commented on PR #10167:
URL: https://github.com/apache/nifi/pull/10167#issuecomment-3156834409

   Made a small change to make sure that we compute the schema of the writer out of the schema of the reader if, and only if, the writer is configured to inherit the schema from the reader. This is not a breaking change as using the reader's schema in extract mode is not a working option today.


-- 
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

Re: [PR] NIFI-8269 - Add support for schema inference in ForkRecord processor when extracting array records [nifi]

Posted by "mattyb149 (via GitHub)" <gi...@apache.org>.
mattyb149 commented on PR #10167:
URL: https://github.com/apache/nifi/pull/10167#issuecomment-3666294435

   Thanks for the explanation! LGTM +1, thanks for this very useful improvement! Merging to main


-- 
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

Re: [PR] NIFI-8269 - Add support for schema inference in ForkRecord processor when extracting array records [nifi]

Posted by "mattyb149 (via GitHub)" <gi...@apache.org>.
mattyb149 merged PR #10167:
URL: https://github.com/apache/nifi/pull/10167


-- 
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