Re: Welcome Anand and Joseph as new committers!

2016-06-10 Thread Brenden Matthews
Welcome and congrats! On Fri, Jun 10, 2016 at 2:11 PM, Benjamin Mahler wrote: > Welcome Anand and Joseph, thanks for all of your contributions! > > Looking forward to seeing your ongoing positive influences on the > community and the project, let's build great software! > >

Re: [VOTE] Release Apache Mesos 0.26.0 (rc5)

2015-12-11 Thread Brenden Matthews
+1 (binding) Make check'd on OS X with SSL & libevent. On Fri, Dec 11, 2015 at 11:11 AM, Benjamin Mahler wrote: > +1 (binding) > > Make check on OS X with and without '--enable-ssl --enable-libevent'. > > On Thu, Dec 10, 2015 at 2:55 PM, Till Toenshoff

Re: Welcome Kapil as Mesos committer and PMC member!

2015-11-05 Thread Brenden Matthews
Awesome! Welcome Kapil! On Thu, Nov 5, 2015 at 5:20 PM, Benjamin Mahler wrote: > Congrats and welcome! > > On Thu, Nov 5, 2015 at 4:04 PM, Kapil Arya wrote: > >> Thanks everyone! :-). >> >> >> >> On Thu, Nov 5, 2015 at 3:50 PM, Elizabeth Lingg

Re: [VOTE] Release Apache Mesos 0.25.0 (rc3)

2015-10-12 Thread Brenden Matthews
+1 (binding) Tested on CI. On Sun, Oct 11, 2015 at 4:12 AM, Michael Park wrote: > +1 (binding) > > Ran *make distcheck* successfully on Ubuntu 14.04 with gcc + clang, CentOS > 7.1 with gcc > Ran *make check* with one non-blocker failure (MESOS-3604 >

Re: [VOTE] Release Apache Mesos 0.22.2 (rc1)

2015-09-28 Thread Brenden Matthews
+1 (binding), tested on internal CI On Thu, Sep 24, 2015 at 8:58 AM, haosdent wrote: > Oh, got it. CentOS 6.6 don't have automake 1.4, so have to build it for > source code. I fetch it from > https://git-wip-us.apache.org/repos/asf/mesos.git before. > > On Thu, Sep 24, 2015

Re: [VOTE] Release Apache Mesos 0.23.1 (rc1)

2015-09-24 Thread Brenden Matthews
+1 (binding) On Thu, Sep 24, 2015 at 3:21 PM, Niklas Nielsen wrote: > +1 (binding) > > Tested on centos7 and ubuntu 14.04 > > On 24 September 2015 at 07:53, Alexander Rojas > wrote: > >> +1 (non binding) >> >> Tested Ubuntu 14.04, OSX >> >> > On

Re: Suggestion: Mesos 0.22.1 point release

2015-03-30 Thread Brenden Matthews
+1 for stability. On Mar 30, 2015 6:26 PM, Benjamin Hindman b...@eecs.berkeley.edu wrote: Obviously a +1, this is a stability fix we should get to our users as soon as possible. On Mon, Mar 30, 2015 at 9:01 PM, Niklas Nielsen nik...@mesosphere.io wrote: Hi all, Joris and Ben H

Re: Mesos Hadoop Framework 0.1.0

2015-03-28 Thread Brenden Matthews
This is great, thanks Tom. On Sat, Mar 28, 2015 at 6:52 AM, Tom Arnfeld t...@duedil.com wrote: Hey everyone, I thought it best to send an email to the list before merging and tagging a 0.1.0 release for the Hadoop on Mesos framework. This release is for a new feature we've been working on

Re: [VOTE] Release Apache Mesos 0.22.0 (rc4)

2015-03-18 Thread Brenden Matthews
+1 Tested with internal testing cluster. On Wed, Mar 18, 2015 at 1:25 PM, craig w codecr...@gmail.com wrote: +1 On Wed, Mar 18, 2015 at 3:52 PM, Niklas Nielsen nik...@mesosphere.io wrote: Hi all, Please vote on releasing the following candidate as Apache Mesos 0.22.0. 0.22.0 includes

Re: 0.21.0-pre Spark latest

2014-10-28 Thread Brenden Matthews
Since we've recently adopted Spark, I'll second Tim's comment. We had an issue with 0.20.1 that was possibly related to Spark[1], so it's important for us to get this stuff fixed in 0.21.0. Tim, can you elaborate on the issues you saw? Have you tested with my recent Spark patches[2][3]? [1]:

Re: [RESULT][VOTE] Release Apache Mesos 0.20.0 (rc2)

2014-08-26 Thread Brenden Matthews
why maven central does not show the jar. Any one has any clue? - Jie On Mon, Aug 25, 2014 at 5:28 PM, Brenden Matthews brenden.matth...@airbedandbreakfast.com wrote: Hey Jie, I'm not seeing the jar in the maven repo: http://search.maven.org/#search%7Cgav%7C1%7Cg%3A%22org.apache.mesos%22

Re: [RESULT][VOTE] Release Apache Mesos 0.20.0 (rc2)

2014-08-25 Thread Brenden Matthews
Hey Jie, I'm not seeing the jar in the maven repo: http://search.maven.org/#search%7Cgav%7C1%7Cg%3A%22org.apache.mesos%22%20AND%20a%3A%22mesos%22 Did something go awry when publishing the jar? On Sun, Aug 24, 2014 at 4:41 PM, Jie Yu yujie@gmail.com wrote: Thanks Dave! On Sun, Aug 24,

Re: Review Request 24569: Added missing 'defer' helpers.

2014-08-11 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24569/#review50238 --- Ship it! This seems to have fixed MESOS-1692. Thanks! - Brenden

Re: [VOTE] Release Apache Mesos 0.18.1 (rc2)

2014-05-02 Thread Brenden Matthews
+1 Make check'd on Debian sid with: gcc version 4.8.2 (Debian 4.8.2-21) On Thu, May 1, 2014 at 11:27 PM, Manivannan citizenm...@gmail.com wrote: +1 On Fri, May 2, 2014 at 11:38 AM, Vinod Kone vinodk...@gmail.com wrote: +1 make check passes on OSX 10.9 w/ gcc-4.8 On Wed, Apr 30, 2014

Re: scaling proposals

2014-05-02 Thread Brenden Matthews
+1 I also second what Tim said. It would be good to record these things into a wiki somewhere, for ease of editing amendment. On Wed, Apr 23, 2014 at 6:22 PM, Tim St Clair tstcl...@redhat.com wrote: Typically the process(es) policies that work best, can be drawn on a simple 'old school'

Re: Review Request 19487: Change linkage around libjvm to use dlopen.

2014-04-23 Thread Brenden Matthews
. - Brenden Matthews On April 23, 2014, 5:09 p.m., Timothy St. Clair wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19487

Re: Review Request 15745: Fixed some task reconciliation cases.

2014-02-27 Thread Brenden Matthews
On Nov. 22, 2013, 8:04 p.m., Niklas Nielsen wrote: Did we get to a conclusion regarding case 1)? and could we write a test which exercises the new scenarios? Brenden Matthews wrote: If I get some time, I'll write a test. I've been testing it in production for a few days though

Re: Scheduler::executorLost() a no-op?

2014-02-20 Thread Brenden Matthews
, Vinod Kone vinodk...@gmail.com wrote: I think this was just overlooked. Feel free to file a bug and send a patch! On Wed, Feb 19, 2014 at 9:33 PM, Brenden Matthews brenden.matth...@airbedandbreakfast.com wrote: Hey folks, I was trying to use the Scheduler::executorLost

Scheduler::executorLost() a no-op?

2014-02-19 Thread Brenden Matthews
Hey folks, I was trying to use the Scheduler::executorLost method[1], but to my dismay I found that the callback was never called. A quick look at the source shows it's not ever referenced in the code, except for a comment mentioning that it perhaps should be[2]. Any thoughts on this? Is the

Re: Review Request 17210: Added 'hostname' flag to the master.

2014-01-24 Thread Brenden Matthews
, 10:42 p.m.) Review request for mesos, Benjamin Hindman, Brenden Matthews, and Jiang Yan Xu. Bugs: MESOS-672 https://issues.apache.org/jira/browse/MESOS-672 Repository: mesos-git Description --- This is part 1 for MESOS-672. Note that this depends

Re: Review Request 17295: Fixed slave to properly do memory isolation when executor resources are not specified.

2014-01-23 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17295/#review32696 --- Ship it! Nice! - Brenden Matthews On Jan. 24, 2014, 2:14 a.m

Re: Review Request 16038: Fixed an exception raised in the master related to task statuses.

2013-12-11 Thread Brenden Matthews
written a patch here: https://github.com/airbnb/mesos/commit/4c47a4eaa3316c892e4551c9ca7703c74aaa56a5 I'm not sure if this was intended or not, however. - Brenden Matthews On Dec. 5, 2013, 8:45 a.m., Ben Mahler wrote

Re: Review Request 16038: Fixed an exception raised in the master related to task statuses.

2013-12-11 Thread Brenden Matthews
On Dec. 11, 2013, 7:35 p.m., Brenden Matthews wrote: Hey Ben, It looks like this might not quite work as desired. I've written a patch here: https://github.com/airbnb/mesos/commit/4c47a4eaa3316c892e4551c9ca7703c74aaa56a5 I'm not sure if this was intended or not, however

Re: mesos and ec2

2013-12-09 Thread Brenden Matthews
My preferred solution is to: 1. Remove the requirement that the forward and reverse DNS match ( https://github.com/airbnb/mesos/commit/6953571758d287d159a289e62c1fd0d685b5e8e3), and 2. Write the leading master's hostname to ZooKeeper for discovery (

Re: Review Request 16038: Fixed an exception raised in the master related to task statuses.

2013-12-05 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/16038/#review29812 --- Ship it! Good catch! - Brenden Matthews On Dec. 5, 2013, 8:45

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-26 Thread Brenden Matthews
On Nov. 22, 2013, 8:04 p.m., Niklas Nielsen wrote: Did we get to a conclusion regarding case 1)? and could we write a test which exercises the new scenarios? Brenden Matthews wrote: If I get some time, I'll write a test. I've been testing it in production for a few days though

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-22 Thread Brenden Matthews
a.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15745/ --- (Updated Nov. 22, 2013, 12:30 a.m

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
--- On Nov. 21, 2013, 12:30 a.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15745/ --- (Updated

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
, visit: https://reviews.apache.org/r/15745/#review29249 --- On Nov. 21, 2013, 12:30 a.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
to any which have had a state change. Review: https://reviews.apache.org/r/15745 Diffs (updated) - src/master/master.cpp a08d01208ff7bbb878b2d50d8406efee4de86171 Diff: https://reviews.apache.org/r/15745/diff/ Testing --- `make check` tested in staging cluster. Thanks, Brenden

Re: Review Request 15745: Fixed some task reconciliation cases.

2013-11-21 Thread Brenden Matthews
On Nov. 21, 2013, 7:59 p.m., Niklas Nielsen wrote: src/master/master.cpp, line 1597 https://reviews.apache.org/r/15745/diff/2/?file=389363#file389363line1597 What happens if status.task_id() does not exist in tasks? Is it silently ignored or does it fail? Brenden Matthews

Review Request 15745: Fixed task reconciliation for lost tasks.

2013-11-20 Thread Brenden Matthews
reconnects to master, and (incorrectly) assumes tasks are still running Diffs - src/master/master.cpp a08d01208ff7bbb878b2d50d8406efee4de86171 Diff: https://reviews.apache.org/r/15745/diff/ Testing --- `make check` tested in staging cluster. Thanks, Brenden Matthews

Review Request 15640: Fixed a bug in Web UI.

2013-11-17 Thread Brenden Matthews
in staging cluster. Thanks, Brenden Matthews

Review Request 15539: Normalized task start/finish time in Web UI.

2013-11-14 Thread Brenden Matthews
--- Tested manually in staging cluster. Thanks, Brenden Matthews

Re: Review Request 14434: Added list of TaskStatuses to Task message.

2013-11-14 Thread Brenden Matthews
. Let's stick to that here too. The '* 1000' in the view is gnarly, but we can move that into the 'isoDate' filter afterward. Brenden Matthews wrote: Oh no, just a few moments too late :) I'll open another review for this. https://reviews.apache.org/r/15539

Re: Review Request 14434: Added list of TaskStatuses to Task message.

2013-11-14 Thread Brenden Matthews
On Nov. 14, 2013, 7:06 p.m., Vinod Kone wrote: src/exec/exec.cpp, line 498 https://reviews.apache.org/r/14434/diff/6/?file=383873#file383873line498 May be only set this if it's not already set by the executor? Silently overwriting is a bit concerning. Brenden Matthews wrote

Re: Review Request 14434: Added list of TaskStatuses to Task message.

2013-11-13 Thread Brenden Matthews
7c243d47d80e7c74fcac938d012b91b67b995490 src/webui/master/static/js/controllers.js 8f65679ff4f82f570cf7d632e45ca7dbcfab3376 Diff: https://reviews.apache.org/r/14434/diff/ Testing --- `make check`, tested in staging, and manual web UI testing. Thanks, Brenden Matthews

Re: Review Request 14434: Added list of TaskStatuses to Task message.

2013-11-13 Thread Brenden Matthews
. - Brenden Matthews On Nov. 13, 2013, 10:17 p.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14434

Re: Review Request 15198: Added a flash service for UI alerts after redirects.

2013-11-06 Thread Brenden Matthews
, for example, only use flash on an opt-in basis. I have it disabled by default unless I need to explicitly enable it. - Brenden Matthews On Nov. 2, 2013, 8:41 p.m., Ross Allen wrote: --- This is an automatically generated e-mail

Re: Review Request 15198: Added a flash service for UI alerts after redirects.

2013-11-06 Thread Brenden Matthews
On Nov. 6, 2013, 8:06 p.m., Brenden Matthews wrote: Looks good, but how does this affect people who don't use flash? I, for example, only use flash on an opt-in basis. I have it disabled by default unless I need to explicitly enable it. Ross Allen wrote: There's no actual

Re: Review Request 14434: Added list of TaskStatuses to Task message.

2013-10-31 Thread Brenden Matthews
6e5cd9f9e48597c7894d6381377c8a291014e8f3 src/webui/master/static/frameworks.html 7c243d47d80e7c74fcac938d012b91b67b995490 Diff: https://reviews.apache.org/r/14434/diff/ Testing --- `make check`, tested in staging, and manual web UI testing. Thanks, Brenden Matthews

Re: Review Request 14948: Corrected reference to $routeParams.

2013-10-25 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14948/#review27555 --- Ship it! - Brenden Matthews On Oct. 25, 2013, 9:48 p.m., Ross

Re: Review Request 14434: Added list of TaskStatuses to Task message.

2013-10-25 Thread Brenden Matthews
/ Testing --- `make check`, tested in staging, and manual web UI testing. Thanks, Brenden Matthews

Re: Review Request 14847: Added --hostname flag to slave and removed webui_hostname.

2013-10-23 Thread Brenden Matthews
On Oct. 22, 2013, 6:01 p.m., Brenden Matthews wrote: It may also be worth looking at these: https://reviews.apache.org/r/11108/ https://reviews.apache.org/r/11975/ This is how we accomplish the same thing. Niklas Nielsen wrote: Hi Brenden, Thanks

Re: Review Request 11975: Use hostname instead of IP for redirection.

2013-10-22 Thread Brenden Matthews
://reviews.apache.org/r/11975/diff/ Testing --- make check Thanks, Brenden Matthews

Re: Review Request 14847: Added --hostname flag to slave and removed webui_hostname.

2013-10-22 Thread Brenden Matthews
On Oct. 22, 2013, 6:01 p.m., Brenden Matthews wrote: It may also be worth looking at these: https://reviews.apache.org/r/11108/ https://reviews.apache.org/r/11975/ This is how we accomplish the same thing. Niklas Nielsen wrote: Hi Brenden, Thanks

Re: Review Request 11108: Simplified os::hostname() in libstout.

2013-10-22 Thread Brenden Matthews
hadoop-2.0.0-mr1-cdh4.2.1 make hadoop-0.20.205.0 make hadoop-0.20.2-cdh3u3 Thanks, Brenden Matthews

Re: Review Request 13599: Use libstout cache for slave completedFrameworks.

2013-10-19 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13599/#review27001 --- On Aug. 14, 2013, 11:18 p.m., Brenden Matthews wrote

Re: Review Request 14434: Added StateTransition to Task message.

2013-10-14 Thread Brenden Matthews
://reviews.apache.org/r/14434/diff/ Testing --- `make check`, tested in staging, and manual web UI testing. Thanks, Brenden Matthews

Re: Review Request 14434: Added timestamps to TaskInfo.

2013-10-04 Thread Brenden Matthews
testing. Thanks, Brenden Matthews

Re: Review Request 14434: Added timestamps to TaskInfo.

2013-10-02 Thread Brenden Matthews
` as mutable? - Brenden --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14434/#review26598 --- On Oct. 2, 2013, 12:18 a.m., Brenden Matthews

Re: Review Request 14434: Added timestamps to TaskInfo.

2013-10-02 Thread Brenden Matthews
. Thanks, Brenden Matthews

Review Request 14434: Added timestamps to TaskInfo.

2013-10-01 Thread Brenden Matthews
in staging, and manual web UI testing. Thanks, Brenden Matthews

[jira] [Resolved] (MESOS-616) The Master / Slave should not store frameworks as both active and completed.

2013-09-23 Thread Brenden Matthews (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-616?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Brenden Matthews resolved MESOS-616. Resolution: Fixed Fix Version/s: 0.14.0 The Master / Slave should not store

[jira] [Commented] (MESOS-688) error of unused_local_typedefs in thirdparty/boost

2013-09-16 Thread Brenden Matthews (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-688?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13768846#comment-13768846 ] Brenden Matthews commented on MESOS-688: https://reviews.apache.org/r/14164

[jira] [Commented] (MESOS-688) error of unused_local_typedefs in thirdparty/boost

2013-09-16 Thread Brenden Matthews (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-688?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13768824#comment-13768824 ] Brenden Matthews commented on MESOS-688: Hmm, I guess old versions of GCC don't

Re: Review Request 14164: Fixed build breakage from 37e946683f1bb6a.

2013-09-16 Thread Brenden Matthews
, visit: https://reviews.apache.org/r/14164/#review26161 --- On Sept. 16, 2013, 10:09 p.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 14164: Fixed build breakage from 37e946683f1bb6a.

2013-09-16 Thread Brenden Matthews
Diffs (updated) - configure.ac 73baf034972549ff34ac1c8f7c098fb3a34baf4e Diff: https://reviews.apache.org/r/14164/diff/ Testing --- make check Thanks, Brenden Matthews

Re: Review Request 14164: Fixed build breakage from 37e946683f1bb6a.

2013-09-16 Thread Brenden Matthews
, visit: https://reviews.apache.org/r/14164/#review26166 --- On Sept. 17, 2013, 3:10 a.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https

[jira] [Commented] (MESOS-688) error of unused_local_typedefs in thirdparty/boost

2013-09-12 Thread Brenden Matthews (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-688?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13765631#comment-13765631 ] Brenden Matthews commented on MESOS-688: FYI: https://reviews.apache.org/r/11974

Re: Building from source

2013-09-11 Thread Brenden Matthews
Hey there, At the moment some patching is required to build with GCC 4.8. The patch you need is available here: https://github.com/airbnb/mesos/commit/e690596ccc41cb21236a54fc9665c8619a0399e5 On Wed, Sep 11, 2013 at 4:19 AM, Humpty Dumpty benevolent...@gmail.comwrote: Hi, I am trying to

Re: Review Request 14039: List modules instead of using package=['.']

2013-09-09 Thread Brenden Matthews
?). Can you double check? - Brenden Matthews On Sept. 9, 2013, 7:04 p.m., Jason Dusek wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14039

Re: Review Request 14057: Empty files are no longer considered as recovery errors.

2013-09-09 Thread Brenden Matthews
staging cluster, and it appeared to work as intended. - Brenden Matthews On Sept. 10, 2013, 2:28 a.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/14057

Re: Review Request 13921: Fixed a bug in reconciliation that failed to account for unknown executors.

2013-08-30 Thread Brenden Matthews
. - Brenden Matthews On Aug. 30, 2013, 11:16 p.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13921/ --- (Updated

Review Request 13892: Fixed broken build in slave_recovery_tests.cpp.

2013-08-28 Thread Brenden Matthews
://reviews.apache.org/r/13892/diff/ Testing --- make check (the tests, however, fail). Thanks, Brenden Matthews

Re: Review Request 13723: Added slave recovery network partition test.

2013-08-28 Thread Brenden Matthews
://gist.github.com/brndnmtthws/a6e5add8e7a09ffdb9cf Here's a build fix: https://reviews.apache.org/r/13892/ - Brenden Matthews On Aug. 22, 2013, 3:51 a.m., Jie Yu wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 13892: Fixed broken build in slave_recovery_tests.cpp.

2013-08-28 Thread Brenden Matthews
this review. - Brenden Matthews On Aug. 28, 2013, 11:05 p.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13892

Re: Review Request 13881: FIx for bad commit 400a88f9817bb102522b08b83dca400380ad8a9b

2013-08-28 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13881/#review25696 --- Ship it! Ship It! - Brenden Matthews On Aug. 28, 2013, 11:02

Re: Review Request 13675: Added a slave load hint for scheduler sorting.

2013-08-26 Thread Brenden Matthews
dc155ba7225ed194525f6ae2a6b96ca8dbdfa432 src/slave/slave.cpp 3e2c6007ca2fbf210a76af376bb5bac5fa32594f Diff: https://reviews.apache.org/r/13675/diff/ Testing --- make check Thanks, Brenden Matthews

Re: Review Request 13744: Fixed a case where Framework re-registration time was not being updated.

2013-08-22 Thread Brenden Matthews
seeing where a framework is marked as 'terminated' even though it's not (according to the web UI)? I keep seeing it with storm (though I have not yet debugged it). - Brenden Matthews On Aug. 22, 2013, 6:37 p.m., Ben Mahler wrote

Re: Review Request 13744: Fixed a case where Framework re-registration time was not being updated.

2013-08-22 Thread Brenden Matthews
On Aug. 22, 2013, 7:39 p.m., Brenden Matthews wrote: This looks good. I wonder if it's related to a bug I'm seeing where a framework is marked as 'terminated' even though it's not (according to the web UI)? I keep seeing it with storm (though I have not yet debugged it). Ben Mahler

Re: Review Request 13675: Added a slave load hint for scheduler sorting.

2013-08-19 Thread Brenden Matthews
d53b8bb97da45834790cca6e04b70b969a8d3453 src/slave/slave.cpp 92a0a7edb46eb3525e193c81129291e405cebaad Diff: https://reviews.apache.org/r/13675/diff/ Testing --- make check Thanks, Brenden Matthews

Re: Review Request 13621: Added upgrade instructions for 0.14.0.

2013-08-16 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13621/#review25247 --- Ship it! Ship It! - Brenden Matthews On Aug. 16, 2013, 7:03 p.m

Re: Review Request 13621: Added upgrade instructions for 0.14.0.

2013-08-16 Thread Brenden Matthews
options? - Brenden Matthews On Aug. 16, 2013, 7:28 p.m., Vinod Kone wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13621

Re: Review Request 13401: Fixed handling of executor history on slaves.

2013-08-14 Thread Brenden Matthews
Thanks, Brenden Matthews

Re: Review Request 13401: Fixed handling of executor history on slaves.

2013-08-14 Thread Brenden Matthews
/slave/slave.cpp cf9f2923fe0e854d457519d700acc951279e1bba Diff: https://reviews.apache.org/r/13401/diff/ Testing --- make check Thanks, Brenden Matthews

Review Request 13599: Use libstout cache for slave completedFrameworks.

2013-08-14 Thread Brenden Matthews
--- Running on staging cluster. `make check` Thanks, Brenden Matthews

[jira] [Commented] (MESOS-616) The Master / Slave should not store frameworks as both active and completed.

2013-08-07 Thread Brenden Matthews (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-616?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13732904#comment-13732904 ] Brenden Matthews commented on MESOS-616: Bump! Also updated the patch here: https

Review Request 13401: Fixed handling of executor history on slaves.

2013-08-07 Thread Brenden Matthews
handling of executor history on slaves. This resolves https://issues.apache.org/jira/browse/MESOS-616. Diffs - src/slave/slave.cpp 3b4911844e19c85a97c79b22ef90897b26f26aac Diff: https://reviews.apache.org/r/13401/diff/ Testing --- make check Thanks, Brenden Matthews

Re: Review Request 13401: Fixed handling of executor history on slaves.

2013-08-07 Thread Brenden Matthews
3b4911844e19c85a97c79b22ef90897b26f26aac Diff: https://reviews.apache.org/r/13401/diff/ Testing --- make check Thanks, Brenden Matthews

Re: [ANNOUNCE] Welcome Ross Allen and David Lester as Apache Mesos PMC and committer!

2013-08-06 Thread Brenden Matthews
Yay! Congrats guys! On Mon, Aug 5, 2013 at 10:34 PM, Mattmann, Chris A (398J) chris.a.mattm...@jpl.nasa.gov wrote: Welcome dudes! ++ Chris Mattmann, Ph.D. Senior Computer Scientist NASA Jet Propulsion Laboratory Pasadena, CA

Re: Review Request 13261: Clarified the guidance for users when slave recovery fails.

2013-08-06 Thread Brenden Matthews
On Aug. 5, 2013, 8:57 p.m., Brenden Matthews wrote: Is there not a better way to handle this process in an automated fashion? It seems to require user intervention if the slave gets in to a bad state. Vinod Kone wrote: Thats a good point. The reason we wanted explicit intervention

[jira] [Commented] (MESOS-616) The Master / Slave should not store frameworks as both active and completed.

2013-08-06 Thread Brenden Matthews (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-616?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13731103#comment-13731103 ] Brenden Matthews commented on MESOS-616: How about something like this: https

Review Request 13211: Fixed bug where active TaskTrackers are killed.

2013-08-01 Thread Brenden Matthews
://reviews.apache.org/r/13211/diff/ Testing --- Used in production at Airbnb. make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Re: Review Request 13077: Updated Mesos mapred-site.xml for Hadoop.

2013-08-01 Thread Brenden Matthews
/ Testing --- Used in production at Airbnb. make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Re: Review Request 12924: Refactor Mesos scheduler into fixed/variable policies.

2013-08-01 Thread Brenden Matthews
://reviews.apache.org/r/13213/ - Brenden Matthews On July 24, 2013, 10:16 p.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12924

Re: Review Request 13211: Fixed bug where active TaskTrackers are killed.

2013-08-01 Thread Brenden Matthews
1923d498121f1ff4e875d9685ed66de2b7fb22ec Diff: https://reviews.apache.org/r/13211/diff/ Testing --- Used in production at Airbnb. make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Review Request 13213: Refactored JobTracker into fixed/variable policies.

2013-08-01 Thread Brenden Matthews
hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Re: Review Request 13212: Allowed JobTracker to use unknown trackers.

2013-08-01 Thread Brenden Matthews
1923d498121f1ff4e875d9685ed66de2b7fb22ec Diff: https://reviews.apache.org/r/13212/diff/ Testing --- make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Re: Review Request 12921: Allowed specification of mininum slot count.

2013-07-30 Thread Brenden Matthews
p.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12921/ --- (Updated July 30, 2013, 7:24 p.m.) Review

Re: Review Request 11119: Don't prematurely kill TaskTrackers.

2013-07-30 Thread Brenden Matthews
://reviews.apache.org/r/9/diff/ Testing --- Used in production at airbnb. make -j10 check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 make hadoop-0.20.205.0 make hadoop-0.20.2-cdh3u3 Thanks, Brenden Matthews

Review Request 13078: Be more intelligent about slot allocation.

2013-07-30 Thread Brenden Matthews
-0.20.205.0 make hadoop-0.20.2-cdh3u3 Thanks, Brenden Matthews

Re: Review Request 11119: Don't prematurely kill TaskTrackers.

2013-07-30 Thread Brenden Matthews
/diff/ Testing --- Used in production at airbnb. make -j10 check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 make hadoop-0.20.205.0 make hadoop-0.20.2-cdh3u3 Thanks, Brenden Matthews

Re: Review Request 13077: Updated Mesos mapred-site.xml for Hadoop.

2013-07-30 Thread Brenden Matthews
://reviews.apache.org/r/13077/diff/ Testing --- Used in production at Airbnb. make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Re: Review Request 13077: Updated Mesos mapred-site.xml for Hadoop.

2013-07-30 Thread Brenden Matthews
--- Used in production at Airbnb. make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

Re: Review Request 11161: No longer accept offers on the same host/port.

2013-07-30 Thread Brenden Matthews
check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 make hadoop-0.20.205.0 make hadoop-0.20.2-cdh3u3 Thanks, Brenden Matthews

Review Request 13080: No longer accept offers on the same host/port.

2013-07-30 Thread Brenden Matthews
/13080/diff/ Testing --- Used in production at airbnb. make -j10 check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 make hadoop-0.20.205.0 make hadoop-0.20.2-cdh3u3 Thanks, Brenden Matthews

Re: Review Request 13080: No longer accept offers on the same host/port.

2013-07-30 Thread Brenden Matthews
to abandon the other change because review board was having difficulties. - Brenden Matthews On July 30, 2013, 9:11 p.m., Brenden Matthews wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Review Request 13077: Updated Mesos mapred-site.xml for Hadoop.

2013-07-30 Thread Brenden Matthews
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13077/#review24283 --- On July 30, 2013, 7:44 p.m., Brenden Matthews wrote

Re: Review Request 13077: Updated Mesos mapred-site.xml for Hadoop.

2013-07-30 Thread Brenden Matthews
://reviews.apache.org/r/13077/diff/ Testing --- Used in production at Airbnb. make check cd hadoop make hadoop-2.0.0-mr1-cdh4.2.1 Thanks, Brenden Matthews

  1   2   >