[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-18 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-972785430 /pulsarbot run-failure-checks -- 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 t

[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-15 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-968639925 /pulsarbot run-failure-checks -- 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 t

[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-14 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-968282373 /pulsarbot run-failure-checks -- 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 t

[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-14 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-968244820 > Should we also add this config to `conf/broker.conf`, `conf/standalone.conf` and releated `document`? Thanks for the reminding, added the config to these files. Pleas

[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-13 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-968188816 > I think `allowAutoUpdateSchema` is better. the boolean type of field have implied the meaning of `is` Nice catch. And it's consistent with previous 'allowXXX' field

[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-13 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-968075522 /pulsarbot run-failure-checks -- 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 t

[GitHub] [pulsar] Jason918 commented on pull request #12786: [Issue 12757][broker] add broker config isAllowAutoUpdateSchema

2021-11-13 Thread GitBox
Jason918 commented on pull request #12786: URL: https://github.com/apache/pulsar/pull/12786#issuecomment-968065973 @codelipenghui PTAL -- 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 specifi