Commit 1 removes a bunch of methods that were unused.
Commit 2 realizes that all of this Table business is in fact redundant and can be removed as SourceExpr::InMem is impossible and thus we can assume we have a DbTable.
Commit 3 moves CrudExpr::Update's fn eval_query logic to its own method, like all the other cases, for readability and consistency.
Commit 4 uses TxMode::unwrap_mut that was introduced in commit 2 in 2 more cases.
Description of Changes
Table
business is in fact redundant and can be removed asSourceExpr::InMem
is impossible and thus we can assume we have aDbTable
.CrudExpr::Update
'sfn eval_query
logic to its own method, like all the other cases, for readability and consistency.TxMode::unwrap_mut
that was introduced in commit 2 in 2 more cases.API and ABI breaking changes
None
Expected complexity level and risk
1