JooHyukKim commented on issue #19866:
URL: https://github.com/apache/pulsar/issues/19866#issuecomment-1552968147
> I am not sure what the right solution is. Given that it sounds like there
is a large diff, it might be more complicated than it is worth.
The required changes being large, it is daunting IMHO. But I am just saying
the verification (what this issue is trying to solve) still seem useful,
especially when users need to rush --failing fast will help reduce time wasted.
👍🏻
So, taking this plan further, when pushing through, we can choose either of
2 possible scenarios below.
### A. Gradual change.
1. Add new configuration option `checkNeededUrlExist`.
2. With usage `CluterData.builder().checkNeededUrlExist(false).build();`.
3. Value of `checkNeededUrlExist` will be `false` by default.
2. Document that you can create cluster safely by enabling
checkNeededUrlExist(true)
3. Document that `checkNeededUrlExist(true)` will be enabled by default at
some point.
4. Slowly fix broken tests.
Pros: breaking changes into bits.
Cons: Harder to revert.
### B. All at once
We can pick up where https://github.com/apache/pulsar/pull/19965 was left
off.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
