It would be a good practice not to modify in-place, but don't think it's a bad practice to modify in-place either. If you think about how this could be used, it might even be cleaner to modify in place, so you don't have to declare more variables.
[ Full content available at: https://github.com/apache/geode/pull/3059 ] This message was relayed via gitbox.apache.org for [email protected]
