-
Notifications
You must be signed in to change notification settings - Fork 12
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
Avoid delete sheet data on writes #67
Comments
Here is my initial thought for what the API might be!
Thoughts on the overall behavior? |
The Sheets API for clearing data accepts a range argument. So I think all 3 of these cases (delete none, delete range, delete all) are feasible! https://developers.google.com/sheets/api/reference/rest/v4/spreadsheets.values/clear |
I like it. Thoughts @archiewood? Also, do we currently support any |
For the moment, all COPY TO options effectively have to be specified via the URL. I think this was just an oversight for an early version though, and it would clearly be helpful, e.g. copy <table_name> to 'https://gsheets/url' (FORMAT gsheet, HEADER True) I think @Alex-Monahan's suggestions make sense, though I think we should consider the DuckDB copy options where they exist: https://duckdb.org/docs/sql/statements/copy.html#copy--to-options
https://duckdb.org/docs/sql/statements/copy.html#csv-options
The overwrite and append flags feel like options we could hijack if we wanted |
I love the idea of reusing parameters! However, I'm not sure exactly how to map the 3 behaviors to those settings. Here is what I'm thinking. Is this intuitive enough?
And we could throw an error if someone tried to run:
|
Well, it turns out those built-in names have some requirements: How about this?
|
Alright, yet another update... It turns out that Right now, I have this working with That work for now? |
Not sure if this needs to be an option or if there is a "smarter" way to decide when to clear sheet and when to not in the codebase.
Reference: #66 (comment)
The text was updated successfully, but these errors were encountered: