-
Notifications
You must be signed in to change notification settings - Fork 209
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
Update all store mut trait methods to take &mut self #1072
Conversation
Change StoreMut methods to return &mut Self
Pull Request Test Coverage Report for Build 4035481981
💛 - Coveralls |
Update AlterTable, IndexMut and Transaction trait methods to take &mut Self and return Result<..>
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.
[nit-picking]
Other than minor comments, it looks good
core/src/executor/alter/table.rs
Outdated
Continue(column_types) => column_types, | ||
Break(column_types) => column_types?, | ||
) -> Result<()> { | ||
let target_columns_defs = match source.as_ref().map(AsRef::as_ref) { |
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.
let target_columns_defs = match source.as_ref().map(AsRef::as_ref) { | |
let target_columns_defs = match source.as_deref() { |
} | ||
} | ||
} | ||
|
||
async fn fetch_vec_rows<T: GStore + GStoreMut>( | ||
async fn fetch_vec_rows<T: GStore>( |
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.
nice catch !
Remove IStoreMut, core only uses StoreMut with &mut self.
Update AlterTable, IndexMut and Transaction trait methods to take &mut self.