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.

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

feat(shell-api): use $currentOp agg. stage for db.currentOp #1395

Merged
merged 6 commits into from
Feb 1, 2023

Conversation

himanshusinghs
Copy link
Contributor

This PR updates the implementation of db.currentOp to make use of $currentOp aggregation stage instead of the command form. There is no change in the output format of the helper. The old arguments are also left intact and are appropriately mapped to relevant options for $currentOp stage (reference taken from the old shell)

Closes:

@himanshusinghs himanshusinghs marked this pull request as ready for review January 30, 2023 12:27
packages/i18n/src/locales/en_US.ts Outdated Show resolved Hide resolved
packages/shell-api/src/database.ts Outdated Show resolved Hide resolved
packages/shell-api/src/database.ts Outdated Show resolved Hide resolved
packages/shell-api/src/database.ts Outdated Show resolved Hide resolved
himanshusinghs and others added 4 commits January 30, 2023 14:17
Co-authored-by: Anna Henningsen <anna.henningsen@mongodb.com>
Co-authored-by: Anna Henningsen <anna.henningsen@mongodb.com>
Co-authored-by: Anna Henningsen <anna.henningsen@mongodb.com>
- currentOp does not use runCommandWithCheck any more and hence pushed
  to exceptions list
@himanshusinghs himanshusinghs changed the title feat(shell-api): use $currentOp aggregation stage for currentOp helper instead of command form feat(shell-api): use $currentOp agg. stage for db.currentOp Feb 1, 2023
@himanshusinghs himanshusinghs merged commit 78ae19a into main Feb 1, 2023
@himanshusinghs himanshusinghs deleted the MONGOSH-1157-currentOp-moved-to-aggregation branch February 1, 2023 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants