Skip to content

server [BQL]: Add simple list syntax for DDL #365

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.

Sign up for GitHub

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

Jump to bottom
Merged
glydr merged 4 commits into from
Aug 5, 2024
Merged

server [BQL]: Add simple list syntax for DDL #365

glydr merged 4 commits into from
Aug 5, 2024

Conversation

@ohsayan
Copy link
Member

@ohsayan ohsayan commented Aug 5, 2024 •
edited
Loading

Before:

CREATE MODEL myspace.mymodel(
    username: string,
    password: string,
    followers: list { type: uint64 }
)
ALTER MODEL myspace.mymodel ADD following { type: list { type: uint64 } }

After:

CREATE MODEL myspace.mymodel(username: string, password: string, followers: [uint64])
ALTER MODEL myspace.mymodel ADD following { type: [uint64] }

Do note that this change is fully backwards compatible as we will continue to support the advanced type definition syntax.

✔️ By submitting this pull request, I agree to the CLA at: https://cla.skytable.io/skytable/skytable

@ohsayan ohsayan added D-server Related to the server D-BlueQL labels Aug 5, 2024
@ohsayan
Copy link
Member Author

ohsayan commented Aug 5, 2024

@glydr r+

glydr
glydr approved these changes Aug 5, 2024
View reviewed changes
Copy link
Collaborator

@glydr glydr left a comment

Choose a reason for hiding this comment

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

📌 PR #365 has been approved by @ohsayan

@glydr glydr merged commit e3e5fd4 into next Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@glydr glydr glydr approved these changes

Assignees

No one assigned

Labels

D-BlueQL D-server Related to the server

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ohsayan @glydr