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

Fixes #1214 - Avoid allocating unused objects on the heap #1215

Merged
merged 2 commits into from Jul 20, 2021
Merged

Fixes #1214 - Avoid allocating unused objects on the heap #1215

merged 2 commits into from Jul 20, 2021

Conversation

keith-ratcliffe
Copy link
Collaborator

No description provided.

@keith-ratcliffe keith-ratcliffe linked an issue Jul 2, 2021 that may be closed by this pull request
DatawaveJexlEngine allocates tons of unused objects on the heap #1214
Closed
jwomeara
jwomeara previously approved these changes Jul 15, 2021
View reviewed changes
@ivakegg ivakegg changed the base branch from release/version3.3 to integration July 15, 2021 19:14
@ivakegg ivakegg dismissed jwomeara’s stale review July 15, 2021 19:14

The base branch was changed.

@hgklohr hgklohr added the accepted accepted for merging into a specific branch label Jul 20, 2021
@keith-ratcliffe keith-ratcliffe merged commit 2bb9b20 into integration Jul 20, 2021
@keith-ratcliffe keith-ratcliffe deleted the issue-1214 branch July 20, 2021 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jwomeara jwomeara jwomeara approved these changes

@apmoriarty apmoriarty apmoriarty approved these changes

@alerman alerman Awaiting requested review from alerman

@billoley billoley Awaiting requested review from billoley

@cawaring cawaring Awaiting requested review from cawaring

@drewfarris drewfarris Awaiting requested review from drewfarris

@hlgp hlgp Awaiting requested review from hlgp

@ivakegg ivakegg Awaiting requested review from ivakegg

@jschmidt10 jschmidt10 Awaiting requested review from jschmidt10

@matthpeterson matthpeterson Awaiting requested review from matthpeterson

Assignees
No one assigned
Labels
accepted accepted for merging into a specific branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

DatawaveJexlEngine allocates tons of unused objects on the heap
4 participants
@keith-ratcliffe @jwomeara @apmoriarty @hgklohr