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

Tracking issue: Stabilize Skyhash 1.0 #148

Closed
11 tasks done
ohsayan opened this issue May 11, 2021 · 3 comments · Fixed by #147
Closed
11 tasks done

Tracking issue: Stabilize Skyhash 1.0 #148

ohsayan opened this issue May 11, 2021 · 3 comments · Fixed by #147
Labels
C-actions Related to actions C-enhancement New feature or request C-Model Related to the native multi-model capabilities of TDB C-performance This relates to performance C-protocol Related to protocols C-reliability This issue/PR relates to reliability C-Stability C-tests This is related to the test suite D-cli D-client Related to the CLI client D-server Related to the server L-Hard Requires a "huge" amount of effort P-high High priority required-improvement This issue or PR addresses an already existing implementation that needs to be revised tracking-issue This is a tracking issue (to be used by maintainers only)
Milestone

Comments

@ohsayan
Copy link
Member

ohsayan commented May 11, 2021

Skyhash is the future protocol that Skytable will use. It saves bandwidth, simplifies client implementation and supports complex structures like nested arrays that can hold multiple types. Skytable 0.5.2 will be the last version to use the Terrapipe protocol and reaches EOL today. This issue tracks the migration from Terrapipe to Skyhash.
Tasks:

This is a long and tedious procedure, but let's make it happen!

@ohsayan ohsayan added C-enhancement New feature or request required-improvement This issue or PR addresses an already existing implementation that needs to be revised D-client Related to the CLI client D-server Related to the server D-libtdb Related to the core library C-performance This relates to performance P-high High priority C-protocol Related to protocols C-tests This is related to the test suite L-Hard Requires a "huge" amount of effort C-reliability This issue/PR relates to reliability C-actions Related to actions D-cli C-Model Related to the native multi-model capabilities of TDB C-Stability tracking-issue This is a tracking issue (to be used by maintainers only) labels May 11, 2021
@ohsayan ohsayan removed the D-libtdb Related to the core library label May 11, 2021
@ohsayan
Copy link
Member Author

ohsayan commented May 11, 2021

Why do we still have that label !?

@ohsayan
Copy link
Member Author

ohsayan commented May 12, 2021

Now that skytable/client-rust#4 has landed it's time to base the tests off the driver, upgrade the cli,... meh, lot of work

@ohsayan
Copy link
Member Author

ohsayan commented May 12, 2021

All test macros and the tests themselves have been updated (and gladly fixed a number of silent bugs)

@ohsayan ohsayan added this to the v0.6.0 milestone May 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-actions Related to actions C-enhancement New feature or request C-Model Related to the native multi-model capabilities of TDB C-performance This relates to performance C-protocol Related to protocols C-reliability This issue/PR relates to reliability C-Stability C-tests This is related to the test suite D-cli D-client Related to the CLI client D-server Related to the server L-Hard Requires a "huge" amount of effort P-high High priority required-improvement This issue or PR addresses an already existing implementation that needs to be revised tracking-issue This is a tracking issue (to be used by maintainers only)
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant