-
Notifications
You must be signed in to change notification settings - Fork 400
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
Low level create table #342
Conversation
… for create table
need to fix test cleanup need to add logic to make create() safer want to return an updated DeltaTable (new state) after a create() is successful
…in the test. Still need to do better error handling in the create() method and figure out how to create a _last_checkpoint and update the DeltaTableState
… how to handle possible errors in the create() method
…sing cloned actions, can now update the DeltaTable state after the commit portion of create() method
added more asserts for test
@Smurphy000 could you help manually resolve the comments you already addressed with follow up commits? That will make it a little bit easier for us to do follow up reviews :) |
updated new DeltaTableMetadata to take in a format option change create method resulting Error type to be a transaction error
as suggested by houqp Co-authored-by: QP Hou <dave2008713@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me overall, great work @Smurphy000 !
…for the data actually written to the delta table from the create method
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Thank you @Smurphy000, I hope you had a good time hacking on rust and delta ;) |
Description
This is the current implementation for a low level create table api
Looking for feedback