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

Mapping UUID as Scylla Driver's type #7

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Daniel-Boll
Copy link
Contributor

No description provided.

…ethods

This commit adds a new class UUIDv4 to the scylladb_wrapper/types/uuid.hpp file. The class has three methods: random, from_string and to_string. The random method generates a new UUIDv4 instance with random values. The from_string method creates a new UUIDv4 instance from a string. The to_string method returns the string representation of the UUIDv4 instance.

The implementation of the UUIDv4 class is in the scylladb_wrapper/types/uuid.cpp file.
feat(index.ts): add runQueryValues function call and comment out other function calls
feat(create-and-select.ts): add id field to insert statement with random UUIDv4 value
feat(index.d.ts): add ScyllaTypeMap and ScyllaType types, add UUIDv4 type to ScyllaDBModule interface, update executeSync method to accept ScyllaTypeAny values as params
fix(addon): fix execute_sync function to bind query parameters and handle empty query
feat(addon): add support for binding UUIDv4 values to queries in execute_sync function
fix(addon): add error handling for invalid query parameter types in execute_sync function
fix(addon): add error handling for empty query in execute_sync function
fix(addon): add error handling for invalid operation types in execute_sync function
@Daniel-Boll Daniel-Boll requested review from rsaz and f0ntana May 15, 2023 22:39
@Daniel-Boll Daniel-Boll self-assigned this May 15, 2023
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.

1 participant