As you mentioned, we are adding a new interface here to make the code testable and avoid using the static FunctionService methods. Are any additional steps required when we add new public API like this? Do we need to add documentation? You mentioned we _could_ make this interface internal. My only concern here is if i were a Geode user and I discovered both FunctionService and FunctionExecutionService in my IDE, it wouldn't be clear to me which to use. Maybe we should make this interface internal until we can fully remove the FunctionService interface in Geode 2.0?
[ Full content available at: https://github.com/apache/geode/pull/2982 ] This message was relayed via gitbox.apache.org for [email protected]
