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

Use CTEs in the updatestock procedures #125

Open
andrei-mart opened this issue Feb 22, 2022 · 0 comments
Open

Use CTEs in the updatestock procedures #125

andrei-mart opened this issue Feb 22, 2022 · 0 comments
Assignees
@andrei-mart
Labels
enhancement New feature or request

Comments

@andrei-mart
Copy link
Contributor

The feature allowing to batch single line updates in stored procedures was removed due to a correctness issue. That significantly impacted the NewOrder operation where this feature was used.
We still can take advantage of batching: there is no correctness issue if single line updates are inside CTEs. Hence we should wrap the update statements into CTEs.
@andrei-mart andrei-mart added the enhancement New feature or request label Feb 22, 2022
@andrei-mart andrei-mart self-assigned this Feb 22, 2022
andrei-mart added a commit that referenced this issue Feb 22, 2022
@andrei-mart
[#125] Use CTEs in the updatestock procedures
bb25276 
YugabuteDB batches update statements if wrapped into CTEs
andrei-mart added a commit that referenced this issue Feb 24, 2022
@andrei-mart
[#125] Use CTEs in geopartitioned schema manager
0c73e99
sonalsagarwal pushed a commit that referenced this issue Jul 6, 2022
@andrei-mart
[#125] Use CTEs in the updatestock procedures
0f27629 
YugabuteDB batches update statements if wrapped into CTEs
sonalsagarwal pushed a commit that referenced this issue Jul 6, 2022
@andrei-mart
[#125] Use CTEs in geopartitioned schema manager
a9041e3
sonalsagarwal added a commit that referenced this issue Sep 29, 2023
@sonalsagarwal @andrei-mart
Conflict fix cte (#142)
9a36b61 
* [#125] Use CTEs in the updatestock procedures
* [#125] Use CTEs in geopartitioned schema manager
* Removing no null from column definitions.
* setting yb_enable_expression_pushdown at connection level

Co-authored-by: Hemant Bhanawat <hbhanawat@yugabyte.com>
Co-authored-by: Andrei Martsinchyk <amartsinchyk@yugabyte.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@andrei-mart andrei-mart

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@andrei-mart