Hi Zixuan 

Regarding the comments you left


> This pip attempts to bypass schema checks and also performs unnecessary 
> policy checks.

The following comments answer your two arguments one by one.
---

Regarding the first argument: attempts to bypass schema checks, you left the 
following comment


> -1, I don't suggest we ignore the replicator check. Once the schema is 
> incompatible, we should disconnect the replicator.


What I wrote in the proposal is "always allow the replicator to register a new 
schema if compatible"; it seems that you did not read the doc carefully


---


Regarding the second argument: performs unnecessary policy checks, you left the 
following comment


> If you want to replicate __change_events, please make sure the topic type is 
> the same between the source and remote clusters.


The replication of `__change_events` is defined by 
PIP-92(https://github.com/apache/pulsar/issues/13444), the current proposal 
only keeps the previous behavior. You can submit a separate discussion to argue 
whether PIP-92 is needed


---


Thanks
Yubiao Feng

Reply via email to