Feature Impl: cost model uses number of requested write locks

[apfitzge]

Problem

Implementation of #34819

Summary of Changes

• Some write-locks are demoted to read-locks during transaction sanitization
• Change the cost model to apply write lock costs for the number of requested write locks, instead of the number of write-locks after demotion
• Requires feature-gate so that activation of 2ry7ygxiYURULZCrypHhveanvP5tzZ4toRwVp89oCNSj (
  Feature Gate: Validator applies cost tracker to blocks during replay #29595) will not cause this change to break consensus

Fixes #

[codecov]

Codecov Report

Attention: 98 lines in your changes are missing coverage. Please review.

Comparison is base (9d13244) 81.7% compared to head (4ea6629) 81.7%.
Report is 14 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #34820     +/-   ##
=========================================
- Coverage    81.7%    81.7%   -0.1%     
=========================================
  Files         826      826             
  Lines      223114   223205     +91     
=========================================
+ Hits       182335   182399     +64     
- Misses      40779    40806     +27     

[tao-stones]

lgtm

[apfitzge]

@taozhu-chicago sorry had to rebase because of conflict in adding features to feature-set

[tao-stones]

lgtm