Skip to content

Very big update to query multi-opensearch - #138

Merged
atheurer merged 6 commits into
masterfrom
multi-opensearch-fixes
Jul 31, 2025
Merged

Very big update to query multi-opensearch#138
atheurer merged 6 commits into
masterfrom
multi-opensearch-fixes

Conversation

@atheurer

Copy link
Copy Markdown
Contributor

-fix xz file reads with differenbt package that
does not leak memory
-lots of minor fixes to query proper month/year for
cdmv9
-fix query that includes >10000 terms (metric data
query for docs partially in the begin-end time range)

-fix xz file reads with differenbt package that
 does not leak memory
-lots of minor fixes to query proper month/year for
 cdmv9
-fix query that includes >10000 terms (metric data
 query for docs partially in the begin-end time range)
@k-rister

Copy link
Copy Markdown
Contributor

Not sure if this is the only problem, but you appear to have broken crucible rm:
image

@k-rister

Copy link
Copy Markdown
Contributor

Looks like crucible rm is still broken.

@atheurer

Copy link
Copy Markdown
Contributor Author

Thanks, will address the rm thing today

k-rister
k-rister previously approved these changes Jul 31, 2025

@k-rister k-rister left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Every time I see the changes in node_modules I find myself wishing we took the time to make all that stuff not be embedded in our repo :(

Comment thread queries/cdmq/add-run.js
Comment thread queries/cdmq/add-run.js
Comment thread queries/cdmq/add-run.js
Comment thread queries/cdmq/add-run.js Outdated
Comment thread queries/cdmq/cdm.js
@atheurer
atheurer merged commit 62db0be into master Jul 31, 2025
20 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Crucible Tracking Jul 31, 2025
@k-rister
k-rister deleted the multi-opensearch-fixes branch July 31, 2025 21:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants