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

[YSQL] Extract Buffering Functionality Into Separate Class #11815

Closed
nathanhjli opened this issue Mar 21, 2022 · 0 comments
Closed

[YSQL] Extract Buffering Functionality Into Separate Class #11815

nathanhjli opened this issue Mar 21, 2022 · 0 comments
Assignees
@d-uspenskiy
Labels
area/ysql Yugabyte SQL (YSQL) kind/enhancement This is an enhancement of an existing feature

Comments

@nathanhjli
Copy link
Contributor

Description

We want to extract buffering functionality into a separate class to support future implementations of advanced flushing where we process multiple buffers instead of just one.
@nathanhjli nathanhjli added the area/ysql Yugabyte SQL (YSQL) label Mar 21, 2022
@nathanhjli nathanhjli assigned nathanhjli and unassigned nathanhjli Mar 21, 2022
@d-uspenskiy d-uspenskiy self-assigned this Mar 22, 2022
@d-uspenskiy d-uspenskiy added the kind/enhancement This is an enhancement of an existing feature label Mar 22, 2022
d-uspenskiy added a commit that referenced this issue Mar 25, 2022
@d-uspenskiy
[#11815] YSQL: Move write operation buffering into separate file
7898813 
Summary: To simplify the code of `PgSession` class and simplify further implementation of write operation buffering subsystem improvements (issue #11628) code related to buffering is moved into separate class `PgOperationBuffer`. Current functionality of buffering subsystem is preserved (in general).

Test Plan: Jenkins

Reviewers: nli, pjain

Reviewed By: pjain

Differential Revision: https://phabricator.dev.yugabyte.com/D16083
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@d-uspenskiy d-uspenskiy

Labels
area/ysql Yugabyte SQL (YSQL) kind/enhancement This is an enhancement of an existing feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@nathanhjli @d-uspenskiy