nifi icon indicating copy to clipboard operation
nifi copied to clipboard

NIFI-8134 allow unescapeJson Record Path function to recursively convert Maps to Records

Open ChrisSamo632 opened this issue 2 years ago • 4 comments

Summary

NIFI-8134 allow unescapeJson Record Path function to recursively convert Maps to Records

Tracking

Please complete the following tracking steps prior to pull request creation.

Issue Tracking

Pull Request Tracking

  • [x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
  • [x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

  • [x] Pull Request based on current revision of the main branch
  • [x] 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

  • [x] Build completed using mvn clean install -P contrib-check
    • [x] JDK 17

Licensing

  • ~[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy~
  • ~[ ] New dependencies are documented in applicable LICENSE and NOTICE files~

Documentation

  • [x] Documentation formatting appears as expected in rendered files

ChrisSamo632 avatar Sep 16 '23 16:09 ChrisSamo632

@markap14 grateful if you've time to re-review this change at some point

ChrisSamo632 avatar Jan 05 '24 07:01 ChrisSamo632

Have rebased from latest main to address merge conflicts, this PR should be ready for re-review @markap14 / @exceptionfactory

ChrisSamo632 avatar Feb 16 '24 20:02 ChrisSamo632

Thanks @ChrisSamo632 will take a look this week

markap14 avatar Feb 20 '24 17:02 markap14

@markap14 thanks for the review, I've addressed your comments (and reverted the IDE auto-formatting issues!)

ChrisSamo632 avatar Feb 28 '24 21:02 ChrisSamo632

Rebased against latest main to refactor changes into new nifi-extension-bundles module

ChrisSamo632 avatar May 04 '24 06:05 ChrisSamo632

Thanks for rebasing @ChrisSamo632 !

joewitt avatar May 04 '24 16:05 joewitt

Thanks for the update @ChrisSamo632 all looks good to me at this point. Sorry this has taken so long, and thanks for sticking with it! +1 will merge to main

markap14 avatar May 14 '24 21:05 markap14