Updated the metrics design to include details on how metrics can be enabled at the request level, client level and global level.

[millems]

No description provided.

[codecov-commenter]

Codecov Report

Merging #1926 into master will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1926      +/-   ##
============================================
+ Coverage     76.24%   76.27%   +0.02%     
  Complexity      187      187              
============================================
  Files          1076     1076              
  Lines         32573    32575       +2     
  Branches       2569     2569              
============================================
+ Hits          24835    24845      +10     
+ Misses         6478     6471       -7     
+ Partials       1260     1259       -1     

Flag	Coverage Δ	Complexity Δ
#unittests	76.27% <ø> (+0.02%)	187.00 <ø> (ø)

Impacted Files	Coverage Δ	Complexity Δ
...nio/netty/internal/OldConnectionReaperHandler.java	81.81% <0.00%> (-9.10%)	0.00% <0.00%> (ø%)
...amodb/internal/operations/DeleteItemOperation.java	94.87% <0.00%> (+0.27%)	0.00% <0.00%> (ø%)
.../http/nio/netty/internal/NettyRequestExecutor.java	74.24% <0.00%> (+5.55%)	0.00% <0.00%> (ø%)

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fc38805...3b8c031. Read the comment docs.

[zoewangg]

Should we make it case insensitive?

[millems]

+1, as we talked about, though, we should use what other booleans do, first and foremost.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
0 Code Smells

No Coverage information
No Duplication information

The version of Java (1.8.0_252) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here