Change BinLog (/bl) to no longer enable Detail Summary (/ds).

[yuehuang010]

Fixes #11244

Context

When using Binary Logger (/bl), it also enables Detail Summary (/ds). It adds 3-5% additional size to the binlog itself.
BL requires diagnostic verbosity that indirectly enables DS.

Changes Made

This change will specifically not enable DS from BL by evaluating the verbosity state before upping the verbosity level. Users can still enable add /ds or /v:diag.

Testing

There isn't a unit test for this area, so manually test it.

Is Detail Summary Shown

Arguments	Before	After
	no	no
/bl	yes	no
/bl /ds	yes	yes
/bl /v:d	yes	no
/bl /v:diag	yes	yes
/fl	no	no
/fl /ds	yes	yes
/fl /v:d	no	no
/fl /v:diag	yes	yes
/bl /fl	yes	no

Notes

[YuliiaKovalova]

cc: @KirillOsenkov

[KirillOsenkov]

See related:

• Allow turning detailed summary off #6338 (comment)
• Binlog causes detailed summary messages output to console #4409

@rainersigwald

[KirillOsenkov]

And yes, as a data point, I've never had the need to look at the detailed summary (or perhaps I've never learned how to read it)

[rainersigwald]

I have used it but not in a long time--I think it's well superseded by other binlog/viewer features.