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

Implement a concurrent hashmap with identity semantics #9449

Closed
ggezer opened this issue Jul 11, 2019 · 0 comments · Fixed by #9597
Closed

Implement a concurrent hashmap with identity semantics #9449

ggezer opened this issue Jul 11, 2019 · 0 comments · Fixed by #9597
Labels
priority-medium type-debt This issue is about tech debt

Comments

@ggezer
Copy link
Contributor

ggezer commented Jul 11, 2019

Summary
gRPC data marshaller has a dependency on an old netty package for `ConcurrentIdentityHashMap' implementation. Replacing that with in-house implementation or importing from a more lightweight package could strip alluxio code of all netty dependencies.
@ggezer ggezer added the type-debt This issue is about tech debt label Jul 11, 2019
@ggezer ggezer mentioned this issue Jul 11, 2019
Merged
ZacBlanco added a commit to ZacBlanco/alluxio that referenced this issue Jul 30, 2019
@ZacBlanco
Implement a ConcurrentIdentityHashMap
09bffe3 
- Implementation of a ConcurrentIdentityHashMap to avoid using
Netty dependencies

Closes Alluxio#9449
@ZacBlanco ZacBlanco mentioned this issue Jul 30, 2019
Merged
alluxio-bot pushed a commit that referenced this issue Aug 7, 2019
@ZacBlanco @alluxio-bot
Implement a ConcurrentIdentityHashMap
ba3f416 
- Implementation of a ConcurrentIdentityHashMap to avoid using
Netty dependencies

- Removed the `io.netty` dependency in the core-common
module. The rest of the dependency cleanup throughout the POMs
is occurring in #9455

Closes #9449

pr-link: #9597
change-id: cid-eb5dfd904018f1c867f8f6d1efc87f586502db0e
jiacheliu3 pushed a commit to jiacheliu3/alluxio that referenced this issue Nov 12, 2019
@ZacBlanco @jiacheliu3
Implement a ConcurrentIdentityHashMap
a117758 
- Implementation of a ConcurrentIdentityHashMap to avoid using
Netty dependencies

- Removed the `io.netty` dependency in the core-common
module. The rest of the dependency cleanup throughout the POMs
is occurring in Alluxio#9455

Closes Alluxio#9449

pr-link: Alluxio#9597
change-id: cid-eb5dfd904018f1c867f8f6d1efc87f586502db0e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
priority-medium type-debt This issue is about tech debt
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Implement a ConcurrentIdentityHashMap ZacBlanco/alluxio
2 participants
@calvinjia @ggezer