zhyyu commented on issue #7169: URL: https://github.com/apache/skywalking/issues/7169#issuecomment-869484799
> OK, get your point this time. Then, this is a bug. It should be fixed like this. > If DBLatencyThresholdsAndWatcher never has a notified data, its `#value` method should always return `null`, rather than `settingsString.get()`. > Also, we should keep initial `settingsString` forever, and fall back to it when we received `delete` event. Make sense? Yes, it make sense. I am thinking add a dynamicSettingsString to fix it. Can I send a pull request? -- 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]
