Return-Path: X-Original-To: apmail-aurora-reviews-archive@minotaur.apache.org Delivered-To: apmail-aurora-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 9B6BA1897A for ; Tue, 24 Nov 2015 19:23:59 +0000 (UTC) Received: (qmail 81709 invoked by uid 500); 24 Nov 2015 19:23:59 -0000 Delivered-To: apmail-aurora-reviews-archive@aurora.apache.org Received: (qmail 81656 invoked by uid 500); 24 Nov 2015 19:23:59 -0000 Mailing-List: contact reviews-help@aurora.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@aurora.apache.org Delivered-To: mailing list reviews@aurora.apache.org Received: (qmail 81633 invoked by uid 99); 24 Nov 2015 19:23:59 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 24 Nov 2015 19:23:59 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id C4FFE2E3DFD; Tue, 24 Nov 2015 19:23:57 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============7457743075467884454==" MIME-Version: 1.0 Subject: Re: Review Request 40656: Remove SessionKey from APIs and implementations. From: "Zameer Manji" To: "Maxim Khutornenko" , "Zameer Manji" Cc: "Bill Farner" , "Aurora" Date: Tue, 24 Nov 2015 19:23:57 -0000 Message-ID: <20151124192357.26797.60746@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Zameer Manji" X-ReviewGroup: Aurora X-Auto-Response-Suppress: DR, RN, OOF, AutoReply X-ReviewRequest-URL: https://reviews.apache.org/r/40656/ X-Sender: "Zameer Manji" References: <20151124171921.26797.76703@reviews.apache.org> In-Reply-To: <20151124171921.26797.76703@reviews.apache.org> Reply-To: "Zameer Manji" X-ReviewRequest-Repository: aurora --===============7457743075467884454== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40656/#review107822 ----------------------------------------------------------- api/src/main/thrift/org/apache/aurora/gen/api.thrift There are two things being done in this review. Removing this struct from our APIs and removing this struct entirely. I beleive if you restore this struct this patch will go green and our APIs will not have SessionKey. We can later remove it from the client side if you don't want to tackle that now. - Zameer Manji On Nov. 24, 2015, 9:19 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/40656/ > ----------------------------------------------------------- > > (Updated Nov. 24, 2015, 9:19 a.m.) > > > Review request for Aurora, Maxim Khutornenko and Zameer Manji. > > > Bugs: AURORA-814 > https://issues.apache.org/jira/browse/AURORA-814 > > > Repository: aurora > > > Description > ------- > > Since our auth* is now transport-level, we don't need it in the app-level messages. > > > Diffs > ----- > > api/src/main/thrift/org/apache/aurora/gen/api.thrift b2628846dfe1d9add0d0edb0e7f2f6ce0121de2b > src/main/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterface.java ccabf55c947393bc04147c8527507e621262e7f3 > src/main/java/org/apache/aurora/scheduler/thrift/aop/AnnotatedAuroraAdmin.java 6d30c25e62061855d8b22d58ad6f33fab7fcc3fb > src/main/python/apache/aurora/client/api/scheduler_client.py c973da109103b0b2f2d47e45a353898f0635203c > src/test/java/org/apache/aurora/scheduler/http/api/ApiBetaTest.java fb5159db0a0e98d0db2fe506d0497f6b4cc09281 > src/test/java/org/apache/aurora/scheduler/http/api/security/HttpSecurityIT.java 2454959eab3334e10e6bea8b7a600b9e88adb83d > src/test/java/org/apache/aurora/scheduler/http/api/security/ShiroAuthorizingInterceptorTest.java 650f21460a3853dc59cec94544125c9c270404b8 > src/test/java/org/apache/aurora/scheduler/http/api/security/ShiroAuthorizingParamInterceptorTest.java d03cd2ae9798819f3ac9b9a563957393c219bb41 > src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java 2bfc2a79aa76a8aed51b428d5d9c2c4e77fca50c > src/test/java/org/apache/aurora/scheduler/thrift/ThriftIT.java 63c20d19a641119e59003b6c31e6b1de3462defb > src/test/java/org/apache/aurora/scheduler/thrift/aop/AopModuleTest.java dafcf25cab9c6974fad07cd7ae271983c919f8c4 > src/test/java/org/apache/aurora/scheduler/thrift/aop/ForwardingThrift.java a6769e62d61b2851db055e98ee254f707a91d208 > src/test/java/org/apache/aurora/scheduler/thrift/aop/LoggingInterceptorTest.java f0caa8bfa4b48aea7ef448243752ef32c8b0c39a > > Diff: https://reviews.apache.org/r/40656/diff/ > > > Testing > ------- > > I did a quick-and-dirty test between a new client and old scheduler to verify that the thrift compatibility works as expected and didn't encounter any problems. > > TODO(wfarner): Verify in end-to-end tests. > > > Thanks, > > Bill Farner > > --===============7457743075467884454==--