Thanks for your proposal, Juan. I still think that it makes sense to remove these warning logs altogether. Even if the stack trace is removed, the amount of logs could still be huge if the operations received is high and the percentage of exceptions significant.
One more factor to add to this discussion is that these warning logs are only generated if the function is HA. If the function returns false to isHA() then the log does not appear. I would say this is one more reason in favor of removing these logs so that the system is consistent. @Barrett Oglesby<mailto:bogle...@vmware.com> are you still in favor of keeping these warning logs? More opinions on this topic are very welcome in order to be able to decide. Thanks, Alberto ________________________________ From: Ju@N <jujora...@gmail.com> Sent: Wednesday, March 30, 2022 7:04 PM To: dev@geode.apache.org <dev@geode.apache.org> Subject: Re: [PROPOSAL] Remove warning logs from FunctionException Hello all, What about something in the middle?: log a WARNING level message stating that the Function named XXX failed and also log the details (including the stack trace) using DEBUG log level?. This would reduce the amount of logs for functions that fail frequently, and will also allow the person troubleshooting/debugging issues to easily tell that something is wrong with function XXX. Cheers. On Wed, 30 Mar 2022 at 17:52, Jacob Barrett <jabarr...@vmware.com> wrote: > > > On Mar 30, 2022, at 9:45 AM, Alberto Gomez <alberto.go...@est.tech<mailto: > alberto.go...@est.tech>> wrote: > > The idea would not be to remove the logs but rather to change the level of > these logs from warning to debug level. > > I agree, if any exception is expected as part user provided action it > should not produce verbose logging. > > -Jake > > -- Ju@N
