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

improve query iterator source management #2694

Open
wants to merge 5 commits into
base: integration
Choose a base branch
from
Open

improve query iterator source management #2694

wants to merge 5 commits into from
+96 −26

Conversation

apmoriarty
Copy link
Collaborator

  • all source deep copies in QueryIterator and extending classes now use the proper method
  • all source deep copy callers provide a stage name that is logged at trace level
  • PipelinePool no longer deep copies a source that is never used

@apmoriarty
Eliminate unnecessary source deep copy when creating a Pipeline
6b34a61
@apmoriarty
All source deep copies in QueryIterator and extending classes now use
e624892 
the preferred synchronized method to ensure thread safety. All callers
now include a stage name which is logged at trace level at the end of
query execution.
@apmoriarty
Merge branch 'integration' into feature/improve-query-iterator-source…
1cda8df 
…-management
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@avgAGB avgAGB avgAGB approved these changes

@alerman alerman alerman approved these changes

Assignees
No one assigned
Labels
Integration Tested linked
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@apmoriarty @alerman @avgAGB