[GitHub] incubator-geode issue #210: GEODE-1647: Add Integrated Security to Peer Auth...

2016-07-20 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/210 I rebased it, and rewrite the GMSAuthenticator. The review is here: https://reviews.apache.org/r/50245/ --- If your project is set up for it, you can reply to this email and have

[GitHub] incubator-geode issue #210: GEODE-1647: Add Integrated Security to Peer Auth...

2016-07-20 Thread kirklund
Github user kirklund commented on the issue: https://github.com/apache/incubator-geode/pull/210 The changes in this PR are incompatible with the changes Jinmei checked in on develop. We'll have to rewrite GMSAuthenticator and its tests. --- If your project is set up for it, you can

[GitHub] incubator-geode issue #210: GEODE-1647: Add Integrated Security to Peer Auth...

2016-07-19 Thread jinmeiliao
Github user jinmeiliao commented on the issue: https://github.com/apache/incubator-geode/pull/210 I'll rebase this onto the current develop, and handle the PR. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your