There is a config in ServiceConfiguration called disableBrokerInterceptors introduced by #8157, which seems to disable the broker interceptor, but is commented for testing only.
Actually, whether to enable the interceptor depends on whether the broker is loaded into the interceptors. #10489 kept the same implementation.
But #20422 has changed the behavior, it uses disableBrokerInterceptors to judge whether to enable the interceptor, which caused an NPE issue mentioned in #20710.
This disableBrokerInterceptors config is very confusing, so we decide to delete it.
Delete config disableBrokerInterceptors.
No backward & forward compatibility issue
- Mailing List discussion thread: https://lists.apache.org/thread/vqf5qcwv3rh2y1r62gw0dnpn0xznq9p0
- Mailing List voting thread: https://lists.apache.org/thread/o11otjlywgd0s8dsv7dg9s8msswdfspp