Skip to content

feat: add explain support for non-cursor commands #2599

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

Merged
merged 26 commits into from
Nov 16, 2020
Merged

feat: add explain support for non-cursor commands #2599

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
a8dfabe
FindOneOperation extends CommandOperation
HanaPearlman Oct 30, 2020
5bded70
implement explain for write commands
HanaPearlman Oct 27, 2020
4374b6b
implement explain for distinct command
HanaPearlman Oct 27, 2020
f00fbfe
implement explain for findAndModify commands
HanaPearlman Oct 27, 2020
1d937a8
implement explain for mapReduce command
HanaPearlman Oct 27, 2020
2a9b1be
first attempt: handle removing sessions for write operations
HanaPearlman Oct 28, 2020
237a70f
consider explain during canRetryWrite
HanaPearlman Oct 28, 2020
c2d0a53
small enum cleanup
HanaPearlman Oct 28, 2020
0126381
model explain after read concern
HanaPearlman Oct 29, 2020
82715bd
create ExplainableCommand class
HanaPearlman Oct 30, 2020
3bf0d78
check explain value in explain command constructor
HanaPearlman Nov 2, 2020
7090c39
quick cursor fix
HanaPearlman Nov 2, 2020
82f0e60
move explain cmd/options to separate file
HanaPearlman Nov 2, 2020
f5e65ed
some commenting
HanaPearlman Nov 3, 2020
4bdf503
respond to comments
HanaPearlman Nov 5, 2020
03333d7
test bug fix
HanaPearlman Nov 9, 2020
20ef5da
Merge branch 'master' into NODE-2852/master/explain-non-cursor
HanaPearlman Nov 9, 2020
c1a4ff9
add explain-related exports to index
HanaPearlman Nov 11, 2020
2c7334b
use aspects and throw from fromOptions
HanaPearlman Nov 12, 2020
d486214
use expanded explain types for clarity
HanaPearlman Nov 12, 2020
ddc9826
change test names and ordering
HanaPearlman Nov 12, 2020
6a17656
clean up
HanaPearlman Nov 12, 2020
7235fb7
Merge branch 'master' into NODE-2852/master/explain-non-cursor
HanaPearlman Nov 12, 2020
001ec2e
fix explain export in index
HanaPearlman Nov 12, 2020
0b37761
check explain supported in individual op classes
HanaPearlman Nov 13, 2020
31bb9eb
Merge branch 'master' into NODE-2852/master/explain-non-cursor
HanaPearlman Nov 13, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix explain export in index
  • Loading branch information
@HanaPearlman
HanaPearlman committed Nov 12, 2020
commit 001ec2e88084b348282780c426d40aff7c97f13a
6 changes: 1 addition & 5 deletions src/index.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -163,11 +163,7 @@ export type {
export type { DbPrivate, DbOptions } from './db';
export type { AutoEncryptionOptions, AutoEncryptionLoggerLevels, AutoEncrypter } from './deps';
export type { AnyError, ErrorDescription } from './error';
export type {
ExplainOptions,
ExplainVerbosity as Verbosity,
ExplainVerbosityLike as VerbosityLike
} from './explain';
export type { ExplainOptions, ExplainVerbosity, ExplainVerbosityLike } from './explain';
export type {
GridFSBucketReadStream,
GridFSBucketReadStreamOptions,
Expand Down