Skip to content

Bugfix. Remove dangerous usage of short-lived AQO memory contexts. #126

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
Merged
merged 1 commit into from
Mar 1, 2023
Merged

Bugfix. Remove dangerous usage of short-lived AQO memory contexts. #126

merged 1 commit into from
Mar 1, 2023

Conversation

danolivo
Copy link
Collaborator

Using such a context we should remember about the risks:

  • Recursion in AQO hooks can induce accidential memory context reset.
  • System routines which we call from the extension, could require more long- lived memory contexts on the outside than our.

@danolivo
Bugfix. Remove dangerous usage of short-lived AQO memory contexts.
92ece85 
Using such a context we should remember about the risks:
* Recursion in AQO hooks can induce accidential memory context reset.
* System routines which we call from the extension, could require more long-
lived memory contexts on the outside than our.
@danolivo danolivo added the bug label Feb 16, 2023
@danolivo danolivo added this to the AQO 1.6 milestone Feb 16, 2023
@danolivo danolivo requested a review from Alena0704 February 16, 2023 04:09
@danolivo danolivo self-assigned this Feb 16, 2023
@danolivo danolivo merged commit 27bff63 into stable14 Mar 1, 2023
@danolivo danolivo deleted the planner-aqo-short-memctx-fix branch March 1, 2023 03:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alena0704 Alena0704 Awaiting requested review from Alena0704

Assignees

@danolivo danolivo

Labels
bug
Projects
None yet
Milestone
AQO 1.6
Development

Successfully merging this pull request may close these issues.
1 participant
@danolivo