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.

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

feat: Delete and Insert operators for structured data #587

Merged
merged 56 commits into from
Feb 17, 2023
Merged

feat: Delete and Insert operators for structured data #587

merged 56 commits into from
Feb 17, 2023

Conversation

jarulraj
Copy link
Member

@jarulraj jarulraj commented Feb 16, 2023 •
edited
Loading

  • Added support for inserting and deleting tuples in structured data storage engine
  • Made batch_mem_size an optional parameter (we could load it just once from the config file at bootstrap time rather than loading it at multiple locations in the codebase) / cc @kaushikravichandran @aryan-rajoria @gaurav274

aryan-rajoria and others added 30 commits February 9, 2023 19:21
@aryan-rajoria
adding delete operation
c9afc12
@aryan-rajoria
Adding Insert Statement
34dfbf7
@aryan-rajoria
checkpoint
9fa9857
@aryan-rajoria
supporting multiple entries
ebe26d3
@aryan-rajoria
implemented for structured data error
bc722dd
@aryan-rajoria
adding parser visitor for delete
0e29858
@aryan-rajoria
delete executor
c1a7864
@aryan-rajoria
delete plan and rules
5ac631a
@aryan-rajoria
adding delete to plan executor
3238e95
@aryan-rajoria
change position of LogicalDelete
02a1d28
@aryan-rajoria
logical delimeter
01181b5
@aryan-rajoria
delete test case
562a7ca
@aryan-rajoria
adding test case
9887732
@aryan-rajoria
adding test case
d2a1a3d
@aryan-rajoria
adding delete testcase
f09c613
@aryan-rajoria
adding predicate to delete executor
79a6168
@aryan-rajoria
adding delete to Image storage
5ce1991
@aryan-rajoria
bug fix in delete
91d7b06
@aryan-rajoria
fixing testcase
0aac934
@aryan-rajoria
adding test case for insert statement
ee48803
@aryan-rajoria
remove order_by from statement_binder.py
fc2f243
@aryan-rajoria
better variable names, using Batch
343a4a2
@aryan-rajoria
error message for insert
121451f
@aryan-rajoria
removing order_by and limit from delete
5b47c15
@aryan-rajoria
remove order_by and limit
8c75a5e
@aryan-rajoria
use f-string
6772cd0
@aryan-rajoria
adding to changelog
7a10d67
@aryan-rajoria
removing commit messages
1a4204f
@aryan-rajoria
formatting
e96d3a4
@aryan-rajoria
fixing comments
640e7ed
@aryan-rajoria aryan-rajoria self-requested a review February 16, 2023 19:24
@jarulraj jarulraj merged commit 86825d3 into master Feb 17, 2023
@jarulraj jarulraj deleted the Aryan-Rajoria-delete-operation branch February 17, 2023 05:55
@jarulraj jarulraj mentioned this pull request Apr 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aryan-rajoria aryan-rajoria Awaiting requested review from aryan-rajoria

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jarulraj @aryan-rajoria