[fix][broker] Fix RetentionPolicies types mismatch.

[Technoboy-]

Fixes #18112

Motivation

Documentation

• doc-not-needed

[coderzc]

Also should modify ManagedLedgerConfig.retentionSizeInMB type.

pulsar/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedgerConfig.java

Lines 437 to 440 in 7b53bd9

	public ManagedLedgerConfig setRetentionSizeInMB(long retentionSizeInMB) {
	this.retentionSizeInMB = retentionSizeInMB;
	return this;
	}

[Technoboy-]

setRetentionSizeInMB

Also should modify ManagedLedgerConfig.retentionSizeInMB type.

pulsar/managed-ledger/src/main/java/org/apache/bookkeeper/mledger/ManagedLedgerConfig.java

Lines 437 to 440 in 7b53bd9

	public ManagedLedgerConfig setRetentionSizeInMB(long retentionSizeInMB) {
	this.retentionSizeInMB = retentionSizeInMB;
	return this;
	}

applyed

[lordcheng10]

LGTM

[codecov-commenter]

Codecov Report

Merging #18114 (afd75b3) into master (6c65ca0) will increase coverage by 11.10%.
The diff coverage is 39.68%.

@@              Coverage Diff              @@
##             master   #18114       +/-   ##
=============================================
+ Coverage     34.91%   46.01%   +11.10%     
- Complexity     5707    17682    +11975     
=============================================
  Files           607     1574      +967     
  Lines         53396   128632    +75236     
  Branches       5712    14159     +8447     
=============================================
+ Hits          18644    59195    +40551     
- Misses        32119    63327    +31208     
- Partials       2633     6110     +3477     

Flag	Coverage Δ
unittests	46.01% <39.68%> (+11.10%)	⬆️

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

Impacted Files	Coverage Δ
.../main/java/org/apache/pulsar/PulsarStandalone.java	0.00% <0.00%> (ø)
.../apache/pulsar/broker/admin/impl/ClustersBase.java	76.66% <0.00%> (+67.62%)	⬆️
...pache/pulsar/broker/admin/impl/NamespacesBase.java	63.25% <ø> (+48.72%)	⬆️
.../pulsar/broker/service/AbstractBaseDispatcher.java	52.14% <0.00%> (+6.28%)	⬆️
...che/pulsar/broker/service/BacklogQuotaManager.java	12.39% <0.00%> (+2.91%)	⬆️
.../pulsar/broker/service/BrokerServiceException.java	44.44% <0.00%> (+19.44%)	⬆️
...ava/org/apache/pulsar/broker/service/Consumer.java	71.63% <0.00%> (+9.62%)	⬆️
...ava/org/apache/pulsar/broker/service/Producer.java	62.72% <0.00%> (+3.08%)	⬆️
...pulsar/broker/service/PulsarCommandSenderImpl.java	74.35% <0.00%> (+7.69%)	⬆️
...va/org/apache/pulsar/broker/service/ServerCnx.java	48.71% <0.00%> (+3.37%)	⬆️
... and 1181 more

[mattisonchao]

Question: Why don't we choose long?

[Technoboy-]

Question: Why don't we choose long?

int is enough.

[nodece]

Question: Why don't we choose long?

int is enough.

I think we still should use the long, long can be compatible with int.

@Technoboy- @mattisonchao