You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by "mkevo (GitHub)" <gi...@apache.org> on 2019/09/10 13:40:49 UTC

[GitHub] [geode] mkevo commented on issue #3765: GEODE-3632 throw NotAuthorizedException for getAll

Hi all!
I agree that this change is wrong. I can try to do something like Jacob's proposal. But the problem is that if we do some for loop to do gets for some keys,  when it cannot get key it will throw exception and go out.
As we have all results of get in List _values_(all exception messages and values which user are available to get), maybe we can at the end get all exception messages from that List and throw NotAuthorizedException with message containing all of these exception messages.

Do you have any other idea? 



[ Full content available at: https://github.com/apache/geode/pull/3765 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org