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

Preparatory work for generic items. #450

Merged
merged 1 commit into from
Jun 7, 2023
Merged

Conversation

leerho
Copy link
Contributor

@leerho leerho commented Jun 7, 2023

  1. The UpdatableBitMask as part of the Flags field has been eliminated. This should make the Flags field identical to the one used in C++.

  2. The documentation of the serialization formats has been significantly improved. See the docs for the KllPreambleUtil class.

  3. I have reduced the dependence on the KllMemoryValidate class to those that are actually required for validation.

  4. More cleanup of fields and variables not really being used.

1. The UpdatableBitMask as part of the Flags field has been eliminated.
This should make the Flags field identical to the one used in C++.

2. The documentation of the serialization formats has been significantly
improved.  See the docs for the KllPreambleUtil class.

3. I have reduced the dependence on the KllMemoryValidate class to those
that are actually required for validation.

4. More cleanup of fields and variables not really being used.
@leerho
Copy link
Contributor Author

leerho commented Jun 7, 2023

I forgot to mention, this also does some preparation of the current master for a ds-java 4.1.0 release.

@leerho leerho merged commit 3b446ee into master Jun 7, 2023
@leerho leerho deleted the prep_for_items_sketch branch June 7, 2023 15:55
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