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

Migrate Hbase.Bytes Class to Gradoop #1269

Closed
galpha opened this issue May 16, 2019 · 2 comments · Fixed by #1407
Closed

Migrate Hbase.Bytes Class to Gradoop #1269

galpha opened this issue May 16, 2019 · 2 comments · Fixed by #1407

Comments

@galpha
Copy link
Contributor

@galpha galpha commented May 16, 2019

Currently, we import the whole hbase-common stack to use its fast implementation of byte handling (PropertyValueStrategies). We should migrate the used class to Gradoop with hints where the files were taken from.

This way we can reduce our jar sizes (and of those who are working with Gradoop in the first place). In addition, it's possible to implement additional features to the bytes class.

@Rascat

This comment has been minimized.

Copy link
Contributor

@Rascat Rascat commented Sep 24, 2019

☝️ i'll look into that

@ChrizZz110

This comment has been minimized.

Copy link
Contributor

@ChrizZz110 ChrizZz110 commented Sep 25, 2019

@Rascat I think @timo95 is already working on that.

timo95 added a commit to timo95/gradoop that referenced this issue Sep 26, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Sep 26, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Sep 26, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Sep 26, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Sep 26, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Sep 30, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Oct 2, 2019
timo95 added a commit to timo95/gradoop that referenced this issue Oct 15, 2019
@galpha galpha closed this in #1407 Oct 16, 2019
galpha added a commit that referenced this issue Oct 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

3 participants
You can’t perform that action at this time.