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

Using reflection to unmonitor a reference results in errors in Java 17 #656

Closed
peter-lawrey opened this issue May 17, 2024 · 1 comment
Closed
Assignees

Comments

@peter-lawrey
Copy link
Member

No description provided.

@peter-lawrey peter-lawrey self-assigned this May 17, 2024
peter-lawrey added a commit that referenced this issue May 20, 2024
* don't use reflection and recursion in IOTools.unmonitor

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes #656

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes #656

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes #656

---------

Co-authored-by: Jerry Shea <jerry.shea@chronicle.software>
@hft-team-city
Copy link
Collaborator

Released in Chronicle-Core-2.25ea15, BOM-2.25ea60

peter-lawrey added a commit that referenced this issue May 28, 2024
* Initial branch of x.26

* Error on deprecated cleanup (#637)

* Review deprecated methods, change build to error on deprecation

* Fix compilation on Java 21

* revert

* Move ByteBuffers to bytes.internal

* Deprecate due x.26

* restore UnsafeMemory.copyMemory

* Remove se.eris.notnull.instrument

* Use a common configuration for the compiler-plugin where deprecation … (#639)

* Use a common configuration for the compiler-plugin where deprecation warnings cause the build to fail

* Remove annotation processor that is more confusing than useful (and often turned off)

* sun.misc.Unsafe usage produces a warning

* review with lint:all (#641)

* Tidy up compiler setting and some warnings

* Revert changes to Jvm.parseSize

* Consolidate references to Thread.currentThread.getId()

* Add uncheckedCast to Jvm

* Add uncheckedCast to Jvm

* Fix deprecated methods/classes in Core (#642)

* Fix deprecated methods/classes in Core

* Add -Xlint:all,-options as default compiler

* Tidy up code base

* resolve Javadoc issues

* Added fail on warning lint for Javadoc, and rawtypes fixes for Bytes (#648)

* review with lint:all (#641)

* Fix javadoc issues

* Fix a rounding error

* Javadoc fixes from x.26 (#650)

* Deprecate due x.26

* Remove se.eris.notnull.instrument

* resolve Javadoc issues

* Fix Javadoc formatting

* Fix test after merge

* Fix warnings following rebase with develop

* Fix warnings following rebase with develop

* Fix newlines

* </p> tag in javadoc not allowed

* Flaky test (#653)

* Give more time for bg thread in flaky test

* Give more time for bg thread in flaky test

* Core/issues/656 (#657)

* don't use reflection and recursion in IOTools.unmonitor

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes #656

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes #656

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes #656

---------

Co-authored-by: Jerry Shea <jerry.shea@chronicle.software>

* Recover from java21 failure (#658)

* Recover from failure on Java21 as mockito isn't supported.

* Recover from failure on Java21 as mockito isn't supported.

* review with lint:all (#641)

* Tidy up compiler setting and some warnings

* Merge with develop

* Merge with develop

* Tidy up compiler setting and some warnings

* remove duplicate

* parent poms -> x.26.0

* Where marked for removal, also mark for deprecation

* Allow for aarch64 CPUs

* This class was left here for backward compatibility.

---------

Co-authored-by: Peter Lawrey <peter.lawrey@chronicle.software>
Co-authored-by: Jerry Shea <jerry.shea@chronicle.software>
peter-lawrey added a commit to OpenHFT/Chronicle-Wire that referenced this issue May 28, 2024
* Initial branch of x.26

* Error on deprecated (#862)

* Review deprecated methods, change build to error on deprecation

* <release>8</release>

* Fix compilation on Java 21

* Don't use UnsafeText here

* Don't use UnsafeText here

* Use release 8 only for Java 11+

* MacOSX issue on modification date

* Clarify why the @deprecated is commented out.

* Clarify why the @deprecated is commented out.

* Deprecate due x.26

* reduced compatibility

* Remove se.eris.notnull.instrument

* Wire to 2.26ea0

* X.26 common compile flags (#865)

* Initial branch of x.26

* Error on deprecated (#862)

* Review deprecated methods, change build to error on deprecation

* <release>8</release>

* Fix compilation on Java 21

* Don't use UnsafeText here

* Don't use UnsafeText here

* Use release 8 only for Java 11+

* MacOSX issue on modification date

* Clarify why the @deprecated is commented out.

* Clarify why the @deprecated is commented out.

* Use a common configuration for the compiler-plugin where deprecation warnings cause the build to fail

* Tidy up compiler setting and some warnings

* X.26 lint all (#868)

* Add -Xlint:all,-options as default compiler

* Add -Xlint:all,-options as default compiler

* Add -Xlint:all,-options as default compiler

* @SuppressWarnings({"rawtypes"}) -> @SuppressWarnings("rawtypes")

* Minor fixes

* Narrow the SuppressWarnings

* X.26 lint all (#869)

* Add -Xlint:all,-options as default compiler

* Add -Xlint:all,-options as default compiler

* Add -Xlint:all,-options as default compiler

* @SuppressWarnings({"rawtypes"}) -> @SuppressWarnings("rawtypes")

* Minor fixes

* Narrow the SuppressWarnings

* Add -Xlint:all,-options as default compiler

* Fix deprecated field

* Fix deprecated field

* Tidy up code base

* resolve merge issues

* resolve Javadoc issues

* Added fail on warning lint for Javadoc, and rawtypes fixes for Bytes (#872)

* serializable warning suppressed

* Fix Javadoc formatting

* Fix warnings following rebase with develop

* Add methods for clear() and emptyHistory() to reduce direct references to VanillaMessageHistory

* Fix warnings following rebase with develop

* removed unwanted cast

* updated test

* FIX deprecation issues in Java 17+

* revert change

* Fix newlines

* </p> tag in javadoc not allowed

* Don't try to serialize a thread (#893)

* Using reflection to unmonitor a reference results in errors in Java 17 Fixes OpenHFT/Chronicle-Core#656

* Fix warnings following rebase with develop

* Add methods for clear() and emptyHistory() to reduce direct references to VanillaMessageHistory

* Handle deprecated members

* parent poms -> x.26.0

* Where marked for removal, also mark for deprecation

---------

Co-authored-by: yevgen.pavlenko <yevgen.pavlenko@chronicle.software>
Co-authored-by: Rob Austin <rob.austin@boundedbuffer.co.uk>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants