[improve][client] PIP-393: Support configuring NegativeAckPrecisionBitCnt while building consumer.

[thetumbled]

Implementation PR for PIP-393: #23601.

Motivation

PR #23600 has implemented most of the logic, but lack of support for configuring NegativeAckPrecisionBitCnt by mistake.

Modifications

Add support for configuring NegativeAckPrecisionBitCnt while building consumer.

Verifying this change

• Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

• Dependencies (add or upgrade a dependency)
• The public API
• The schema
• The default values of configurations
• The threading model
• The binary protocol
• The REST endpoints
• The admin CLI options
• The metrics
• Anything that affects deployment

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

Matching PR in forked repository

PR in forked repository: thetumbled#68

[crossoverJie]

LGTM, Could you please add unit test for client?

[poorbarcode]

How about adding a test for this change?

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 66.66667% with 1 line in your changes missing coverage. Please review.

Project coverage is 74.24%. Comparing base (bbc6224) to head (a959d0c).
Report is 832 commits behind head on master.

Files with missing lines	Patch %	Lines
...apache/pulsar/client/impl/ConsumerBuilderImpl.java	66.66%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master   #23804      +/-   ##
============================================
+ Coverage     73.57%   74.24%   +0.66%     
+ Complexity    32624    31819     -805     
============================================
  Files          1877     1853      -24     
  Lines        139502   143453    +3951     
  Branches      15299    16290     +991     
============================================
+ Hits         102638   106504    +3866     
+ Misses        28908    28576     -332     
- Partials       7956     8373     +417     

Flag	Coverage Δ
inttests	26.80% <0.00%> (+2.22%)	⬆️
systests	23.16% <0.00%> (-1.16%)	⬇️
unittests	73.76% <66.66%> (+0.92%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
...apache/pulsar/client/impl/ConsumerBuilderImpl.java	86.66% <66.66%> (-0.28%)	⬇️

... and 1030 files with indirect coverage changes

[thetumbled]

How about adding a test for this change?

Added.

[lhotari]

@thetumbled Please don't add cherry-picked labels before a change has been cherry-picked. I can handle the cherry-picking for this PR. There are some details about the process at https://pulsar.apache.org/contribute/maintenance-process/ .