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.

Sign up for GitHub

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

Jump to bottom

Add support for Records in JDK 14 #766

Merged
merged 24 commits into from
Mar 18, 2021
Merged

Add support for Records in JDK 14 #766

Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
370aad5
add test and serializer skeleton and update pom
Aug 26, 2020
7dae878
Merge branch 'master' into records
Sep 1, 2020
57822d2
add record serializer and test
Sep 1, 2020
054adcf
minimal commit to reproduce pom config
Sep 2, 2020
e46c198
fix test compilation
Sep 2, 2020
f167b17
fix compile and test run configuration
Sep 3, 2020
59d7532
add record serializer and test and update pom
Sep 3, 2020
f97e3b3
fix lower bound for compiler profile
Sep 3, 2020
2a7c2bf
adjust config and small clean ups
Sep 7, 2020
2d1497c
fix indentation
Sep 7, 2020
48809c9
merge with upstream
Sep 7, 2020
da5a8c2
change to 2-arg constructor to work with JDK14
Sep 7, 2020
b3ac7de
Merge branch 'records' of github.com:oracle/kryo into records
Sep 9, 2020
7641fca
add test data for JDK14 and add ordering by name to serializer
Sep 9, 2020
5ad3f11
remove child pom config (again)
Sep 9, 2020
260ccca
remove .ser files and cleanup RecordSerializer
Sep 9, 2020
04dbbe8
fix typo and empty line
Sep 9, 2020
c66346d
add .ser files and add convenience method
Sep 15, 2020
c7be874
move jdk14 tests and add with build-helper-maven-plugin
Sep 18, 2020
9d7ff95
consolidate jdk11 and jdk14 tests, cleanup imports and debugging output
Sep 23, 2020
34fbe7d
merge in master
Mar 9, 2021
e5e6422
add author tags
Mar 9, 2021
18df838
update ser test/resources and serializer count
Mar 9, 2021
de170a7
fix serializier count
Mar 9, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/com/esotericsoftware/kryo/unsafe/UnsafeUtil.java
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
/* Copyright (c) 2008-2020, Nathan Sweet
* Copyright (C) 2020, Oracle and/or its affiliates.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without modification, are permitted provided that the following
Expand Down Expand Up @@ -96,7 +97,7 @@ public class UnsafeUtil {
static {
ByteBuffer buffer = ByteBuffer.allocateDirect(1);
try {
directByteBufferConstructor = buffer.getClass().getDeclaredConstructor(long.class, int.class, Object.class);
directByteBufferConstructor = buffer.getClass().getDeclaredConstructor(long.class, int.class);
directByteBufferConstructor.setAccessible(true);
} catch (Exception ex) {
if (DEBUG) debug("kryo", "No direct ByteBuffer constructor is available.", ex);
Expand Down Expand Up @@ -124,7 +125,7 @@ public static ByteBuffer newDirectBuffer (long address, int size) {
if (directByteBufferConstructor == null)
throw new UnsupportedOperationException("No direct ByteBuffer constructor is available.");
try {
return directByteBufferConstructor.newInstance(address, size, null);
return directByteBufferConstructor.newInstance(address, size);
} catch (Exception ex) {
throw new KryoException("Error creating a ByteBuffer at address: " + address, ex);
Copy link

Choose a reason for hiding this comment

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

This change is unrelated to records, but is clearly required to successfully run on JDK 14+, since the non-Public constructors of DirectByteBuffer has been refactored / changed.

}
Expand Down