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

Replace Agrona with JCTools as it's Java 7 compatible #101

Merged
merged 1 commit into from
Jun 19, 2018

Conversation

felixbarny
Copy link
Member

@felixbarny felixbarny commented Jun 19, 2018

As an additional benefit the Atomic variants of JCTools' queues don't depend on sun.misc.Unsafe, making it Java 11 proof.

Signed-off-by: Felix Barnsteiner <felix.barnsteiner@elastic.co>
@codecov-io
Copy link

Codecov Report

Merging #101 into master will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #101   +/-   ##
=========================================
  Coverage     76.95%   76.95%           
  Complexity      853      853           
=========================================
  Files            92       92           
  Lines          2973     2973           
  Branches        246      246           
=========================================
  Hits           2288     2288           
  Misses          546      546           
  Partials        139      139
Impacted Files Coverage Δ Complexity Δ
...ain/java/co/elastic/apm/impl/ElasticApmTracer.java 77.48% <100%> (ø) 41 <0> (ø) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f04cd61...ba0ff8f. Read the comment docs.

@felixbarny felixbarny merged commit 6d9f7a1 into elastic:master Jun 19, 2018
@felixbarny felixbarny deleted the jctools branch June 19, 2018 15:19
@v1v v1v mentioned this pull request Mar 7, 2022
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

Successfully merging this pull request may close these issues.

2 participants