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

Import cht source files for better integration #77

Merged
merged 5 commits into from
Feb 1, 2022
Merged

Conversation

tatsuya6502
Copy link
Member

This PR adds source files imported from moka-cht crate.

Changes

  • Add source files (modules) used by Moka (as moka::cht::* modules).
  • Remove moka-cht crate from the dependency.
  • Add crossbeam-epoch v0.8.2 to the dependency.
  • Update the existing use statement in Moka modules to refer to moka::cht::SegmentedHashMap.
  • Change visibility of moka::cht::* modules from public to crate public.
  • Remove unused feature num_cpu from moka::cht::* modules.
  • Temporary disable unit tests for cht::* modules.

Remaining Action Items

  • Update and re-enable unit tests for moka::cht::* modules.
  • Update the doc.

- Add source files (modules) used by Moka (as `moka::cht::*` modules).
- Remove moka-cht crate from the dependency.
- Add crossbeam-epoch v0.8.2 to the dependency.
- Update the existing `use` statement in Moka modules to refer to
  `moka::cht::SegmentedHashMap`.
- Temporary disable unit tests for `cht::*` modules.
- Change visibility of `moka::cht::*` modules from public to crate public.
- Remove unused feature `num_cpu` from `moka::cht::*` modules.
@tatsuya6502 tatsuya6502 added the enhancement New feature or request label Feb 1, 2022
@tatsuya6502 tatsuya6502 added this to the v0.7.2 milestone Feb 1, 2022
- Remove `GlobalStats`, which was added by us (Moka authors) and used
  for performance testing.
- Move the Credit section in the README up above the License section.
@tatsuya6502 tatsuya6502 changed the title Import cht Import cht source files for better integration Feb 1, 2022
@tatsuya6502 tatsuya6502 marked this pull request as ready for review February 1, 2022 10:12
Copy link
Member Author

@tatsuya6502 tatsuya6502 left a comment

Choose a reason for hiding this comment

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

Merging.

@tatsuya6502 tatsuya6502 merged commit d4d9d9c into master Feb 1, 2022
@tatsuya6502 tatsuya6502 deleted the import-cht branch February 1, 2022 10:18
@tatsuya6502 tatsuya6502 self-assigned this Feb 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant