nishita-09 commented on PR #1000: URL: https://github.com/apache/flink-kubernetes-operator/pull/1000#issuecomment-4198041059
> I have added some notes, overall I think there is too much logic in the configmanager. We should move the getting/accessing of the configs to the Observers, then the FlinkConfigManager should simply always apply when available. Sure @gyfora , will make this change. > This is going to be very critical logic, so the exception handling is critical, it's better to fall back to the previous behaviour (simply use the previous logic if the runtime config cannot be accessed) so that part is good (has to go to the observer too). Yes makes sense, even I was of the same opinion. Will take this up. > > Once we restructured this, we should add detailed test coverage for the core logic and error handling :) Thanks @gyfora for the review. Will take a look at this and complete this by EOW. -- 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]
