Hi Geode devs, This PR <https://github.com/apache/geode/pull/4135> makes small changes to the execution times recorded by function stats. I wanted to run this by the dev list to make sure this will not cause problems for users who rely on these stats:
- In ServerFunctionExecutor and ServerRegionFunctionExecutor, we are now including validateExecution in the total execution time. - In MemberFunctionStreamingMessage, we are now including the code leading up to creating the function context in the total execution time. This would potentially make times reported by function stats appear longer than prior to this change. Please reply or comment on the PR if anyone sees a problem with this. Thanks, Aaron