Skip to content
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

ZOOKEEPER-3739 Remove use of com.sun.nio.file.SensitivityWatchEventModifier #1340

Closed
wants to merge 1 commit into from

Conversation

anmolnar
Copy link
Contributor

3.5 backport of #1269

Remove unsupported use of com.sun.nio.file.SensitivityWatchEventModifier
to better support builds against newer JDKs.

Also update build tooling to use strict JDK release compatibility when
building with newer JDKs by adding profiles which automatically activate
the correct compiler flag when the newer JDK is detected when building
with Maven or Eclipse.

Author: Christopher Tubbs <ctubbsii@apache.org>

Reviewers: Enrico Olivelli <eolivelli@apache.org>, Norbert Kalmar <nkalmar@apache.org>

Closes apache#1269 from ctubbsii/ZK-3739
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
Please note that the fix does not affect any build.
But I think we can ignore ant build as we are using Maven for releases

@symat
Copy link
Contributor

symat commented Apr 28, 2020

LGTM +1

@symat
Copy link
Contributor

symat commented Apr 28, 2020

I'll merge it now

asfgit pushed a commit that referenced this pull request Apr 28, 2020
…odifier

3.5 backport of #1269

Author: Christopher Tubbs <ctubbsii@apache.org>

Reviewers: Andor Molnar <andor@apache.org>, Enrico Olivelli <eolivelli@apache.org>, Mate Szalay-Beko <symat@apache.org>

Closes #1340 from anmolnar/ZOOKEEPER-3215_35
@symat
Copy link
Contributor

symat commented Apr 28, 2020

merged to branch-3.5. I also added the 3.5.8 tag to the original Jira. Thank you all!

@symat symat closed this Apr 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants