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] Move setting of set_force_catalog_modifications from PgDmlWrite into PgSession #25434

Open
1 task done
d-uspenskiy opened this issue Dec 26, 2024 · 0 comments
Open
1 task done

[YSQL] Move setting of set_force_catalog_modifications from PgDmlWrite into PgSession #25434

d-uspenskiy opened this issue Dec 26, 2024 · 0 comments
Assignees
@d-uspenskiy
Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage

Comments

@d-uspenskiy
Copy link
Contributor

d-uspenskiy commented Dec 26, 2024 •
edited by jira bot
Loading

Jira Link: DB-14670

Description

The PgDmlWrite class should not care much about current session state. It is reasonable to set set_force_catalog_modifications field in write request straight at sending time by the PgSession class.

Issue Type

kind/bug

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@d-uspenskiy d-uspenskiy added area/ysql Yugabyte SQL (YSQL) status/awaiting-triage Issue awaiting triage labels Dec 26, 2024
@d-uspenskiy d-uspenskiy self-assigned this Dec 26, 2024
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Dec 26, 2024
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/bug This issue is a bug priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yugabyte-ci @d-uspenskiy