[fix][broker] Don't allow creating a non-persistent partitioned topic with '-partition-' in name

[hanmz]

Motivation

Currently, we can create a non-persistent partition topic contain -partition-. However, we cannot use it as a normal non-persistent partition topic. For example, when we use the partitioned-stats command, we will encounter errors.

Partitioned Topic Name should not contain '-partition-'
Reason: Partitioned Topic Name should not contain '-partition-'

Modifications

We should disable the creation of non-persistent partition topics contain -partition-.

Documentation

Verifying this change

All unit tests passed, not have any behavior change.

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

[lhotari]

Thanks for the contribution, @hanmz. Would you be able to add a test case to cover this change?

[hanmz]

Thanks for the contribution, @hanmz. Would you be able to add a test case to cover this change?

OK, I've added a test case.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 32.15%. Comparing base (bbc6224) to head (fbe1f32).
Report is 704 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (bbc6224) and HEAD (fbe1f32). Click for more details.

HEAD has 2 uploads less than BASE

Flag	BASE (bbc6224)	HEAD (fbe1f32)
unittests	2	0

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #23488       +/-   ##
=============================================
- Coverage     73.57%   32.15%   -41.42%     
+ Complexity    32624       86    -32538     
=============================================
  Files          1877     1800       -77     
  Lines        139502   139828      +326     
  Branches      15299    15388       +89     
=============================================
- Hits         102638    44962    -57676     
- Misses        28908    88289    +59381     
+ Partials       7956     6577     -1379     

Flag	Coverage Δ
inttests	27.24% <100.00%> (+2.66%)	⬆️
systests	24.34% <0.00%> (+0.01%)	⬆️
unittests	?

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

Files with missing lines	Coverage Δ
...he/pulsar/broker/admin/v2/NonPersistentTopics.java	29.20% <100.00%> (-32.27%)	⬇️

... and 1627 files with indirect coverage changes

[lhotari]

Closing and reopening to trigger a new CI run