New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NIFI-11269 Added snappy dependency #7035
Conversation
Hi can someone look at this and verify its good? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the contribution @johnAlJunior.
One minor note, the explicit version number for snappy-java
should be removed because it is provided from the project root Maven configuration.
Otherwise, it looks like this change aligns with the inclusion of the snappy-java library in nifi-record-serialization-services
for the AvroReader.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@johnAlJunior If you can remove the version number and rebase the pull request so that it does not contain merge commits, it should be ready to go.
Hi I removed the version, but I don't know how to rebase the pull request... |
8188ddd
to
c99fca4
Compare
Done it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for removing the version @johnAlJunior, the latest version looks good. I can cherry pick the necessary commits.
For future reference, merge commits should be avoided in pull requests, instead, changes from the main
branch be rebased on to the branch, and then the pull request branch can be pushed, or force-pushed if necessary. An article from Atlassian has some useful background on merging vs. rebasing.
Thanks! Will do from now... Happy to contribute! |
This closes #7035 Signed-off-by: David Handermann <exceptionfactory@apache.org>
This closes apache#7035 Signed-off-by: David Handermann <exceptionfactory@apache.org>
Summary
NIFI-11269
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking
Pull Request Tracking
NIFI-00000
NIFI-00000
Pull Request Formatting
main
branchVerification
Please indicate the verification steps performed prior to pull request creation.
Build
mvn clean install -P contrib-check
Licensing
LICENSE
andNOTICE
filesDocumentation