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 167A810BA1 for ; Sat, 21 Feb 2015 01:29:58 +0000 (UTC) Received: (qmail 20194 invoked by uid 500); 21 Feb 2015 01:29:58 -0000 Delivered-To: apmail-aurora-reviews-archive@aurora.apache.org Received: (qmail 20142 invoked by uid 500); 21 Feb 2015 01:29:57 -0000 Mailing-List: contact reviews-help@aurora.incubator.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@aurora.incubator.apache.org Delivered-To: mailing list reviews@aurora.incubator.apache.org Received: (qmail 20131 invoked by uid 99); 21 Feb 2015 01:29:57 -0000 Received: from nike.apache.org (HELO nike.apache.org) (192.87.106.230) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 21 Feb 2015 01:29:57 +0000 X-ASF-Spam-Status: No, hits=-1997.8 required=5.0 tests=ALL_TRUSTED,HTML_MESSAGE,T_RP_MATCHES_RCVD X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO mail.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with SMTP; Sat, 21 Feb 2015 01:29:35 +0000 Received: (qmail 20066 invoked by uid 99); 21 Feb 2015 01:29:31 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Sat, 21 Feb 2015 01:29:31 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 0B2231D2395; Sat, 21 Feb 2015 01:29:30 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============5195732327918764095==" MIME-Version: 1.0 Subject: Re: Review Request 31248: Export stats for source and reason for LOST tasks, and status delivery latency. From: "Bill Farner" To: "Maxim Khutornenko" , "Joshua Cohen" Cc: "Bill Farner" , "Aurora" Date: Sat, 21 Feb 2015 01:29:30 -0000 Message-ID: <20150221012930.21354.11640@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org/ Auto-Submitted: auto-generated Sender: "Bill Farner" X-ReviewGroup: Aurora X-ReviewRequest-URL: https://reviews.apache.org/r/31248/ X-Sender: "Bill Farner" References: <20150221005342.21354.47365@reviews.apache.org> In-Reply-To: <20150221005342.21354.47365@reviews.apache.org> Reply-To: "Bill Farner" X-ReviewRequest-Repository: aurora X-Virus-Checked: Checked by ClamAV on apache.org --===============5195732327918764095== MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit > On Feb. 21, 2015, 12:53 a.m., Joshua Cohen wrote: > > Can you fill in testing done? Honest question - do you find that useful for changes like this? I find it redundant to always type `./gradlew build -Pq`, especially since the build bot will do that anyhow. > On Feb. 21, 2015, 12:53 a.m., Joshua Cohen wrote: > > src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java, lines 199-200 > > > > > > Use StringBuilder and String.format? See my reply to Maxim's comment. javac is smart about using StringBuilder behind the scenes in cases like this. Do you find String.format more readable in cases like this? Personally i do not. > On Feb. 21, 2015, 12:53 a.m., Joshua Cohen wrote: > > src/main/java/org/apache/aurora/scheduler/mesos/TaskStatusStats.java, line 44 > > > > > > s/loast/lost Fixed. > On Feb. 21, 2015, 12:53 a.m., Joshua Cohen wrote: > > src/test/java/org/apache/aurora/scheduler/mesos/TaskStatusStatsTest.java, line 85 > > > > > > move these to previous line (here, elsewhere)? Done. - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/31248/#review73353 ----------------------------------------------------------- On Feb. 21, 2015, 1:11 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/31248/ > ----------------------------------------------------------- > > (Updated Feb. 21, 2015, 1:11 a.m.) > > > Review request for Aurora, Joshua Cohen and Maxim Khutornenko. > > > Bugs: AURORA-1028 > https://issues.apache.org/jira/browse/AURORA-1028 > > > Repository: aurora > > > Description > ------- > > Export stats for source and reason for LOST tasks, and status delivery latency. > > > Diffs > ----- > > src/main/java/org/apache/aurora/scheduler/events/PubsubEvent.java 1d8f0128732756db74576ee669f6a2718fecc105 > src/main/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImpl.java ffc30bb548706df7bec9e1502242890e9b5eb942 > src/main/java/org/apache/aurora/scheduler/mesos/SchedulerDriverModule.java 59ad9e65589c421cefb76f265446fa2885e6198c > src/main/java/org/apache/aurora/scheduler/mesos/TaskStatusStats.java PRE-CREATION > src/test/java/org/apache/aurora/scheduler/mesos/MesosSchedulerImplTest.java d02c6b32841d5d39c5780e7a044079a38729effb > src/test/java/org/apache/aurora/scheduler/mesos/TaskStatusStatsTest.java PRE-CREATION > > Diff: https://reviews.apache.org/r/31248/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > > --===============5195732327918764095==--