Uh oh!
There was an error while loading. Please reload this page.
feat: support partitioned dml in dbapi - #1103
Conversation
| @check_not_closed | ||
| def run_partition(self, encoded_partition_id): | ||
| def run_partition(self, parsed_statement: ParsedStatement): |
There was a problem hiding this comment.
This is a breaking change of a public method
| ) | ||
| @check_not_closed | ||
| def set_autocommit_dml_mode( |
There was a problem hiding this comment.
I think that it would make more sense to have a public method that takes a AutocommitDmlMode enum value as an argument, rather than one that takes a parsed statement as an argument. This method looks like a private method that should only be called by our own parser.
Instead, we should:
- Have a private method that is basically the same as this.
- Have a public method that takes an
AutocommitDmlModeenum as an input argument and that actually changes the value of the flag. That method would then also be usable for anyone using the dbapi driver programmatically, as they could just call that method to change the value.
See https://github.com/googleapis/java-spanner-jdbc/blob/910a130a02f72c4d8764f12e347f6c3d1bd51b2b/src/main/java/com/google/cloud/spanner/jdbc/CloudSpannerJdbcConnection.java#L136 for how the API in the JDBC driver looks.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
No description provided.