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

dbWriteTable cannot be nested within a transaction #41

Closed
ggrothendieck opened this issue Sep 5, 2014 · 1 comment
Closed

dbWriteTable cannot be nested within a transaction #41

ggrothendieck opened this issue Sep 5, 2014 · 1 comment

Comments

@ggrothendieck
Copy link

dbWriteTable uses dbBegin ... dbCommit but these do not nest so dbWriteTable cannot be within a larger transaction.

These should be changed to use SQLite savepoints as those can be nested.

dbGetQuery(con, "savepoint dbWriteTable")
....
dbGetQuery(con, "release dbWriteTable")

For more info see: http://sqlite.org/lang_transaction.html
hadley added a commit that referenced this issue Sep 5, 2014
@hadley
Add named argument to transaction functions.
a028da4 
#41
@hadley hadley closed this as completed in f428333 Sep 5, 2014
troels pushed a commit to troels/RSQLite that referenced this issue Nov 27, 2019
@krlmlr
Merge branch 'list_fields', closes r-dbi#41
09aa953 
- Implement `dbListFields()` (r-dbi#41, @thrasibule).
troels pushed a commit to troels/RSQLite that referenced this issue Nov 27, 2019
@krlmlr
Merge tag 'v0.1-7'
5446433 
- All tests pass again.
- Default to `row.names = FALSE`.
- Fix escaping of `NA` values and character data (@jimhester).
- Consistent coding style.
- Fix `dbDataType()` for data frames (r-dbi#133, @etiennebr).
- Add support for dates and times (r-dbi#111, @thrasibule).
- `dbWriteTable()` gains `field.types` argument (r-dbi#45, @robertzk).
- Implement `dbListFields()` (r-dbi#41, @thrasibule).
- Fix compiler warnings.
troels pushed a commit to troels/RSQLite that referenced this issue Nov 27, 2019
@krlmlr
dbListFields() implemented in r-dbi#41, thanks again @thrasibule! Closes
95a0d96 
 r-dbi#79
troels pushed a commit to troels/RSQLite that referenced this issue Nov 27, 2019
@krlmlr
Merge tag 'v0.1-8'
cc596c9 
- Improve DBI compatibility (r-dbi#24, r-dbi#41, r-dbi#52, r-dbi#53, r-dbi#61, r-dbi#62, r-dbi#70, r-dbi#79, r-dbi#104, r-dbi#121).
- Define `timegm` for Windows and add missing function (r-dbi#136).
- The time zone is set by executing `SET TIMEZONE='UTC'` instead of setting an environment variable (r-dbi#136).
@github-actions
Copy link

This old thread has been automatically locked. If you think you have found something related to this, please open a new issue and link to this old issue if necessary.

@github-actions github-actions bot locked and limited conversation to collaborators Dec 11, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@ggrothendieck