Jenkins build is back to normal : Mesos-Trunk-Ubuntu-Build-Out-Of-Src-Set-JAVA_HOME #1473

2013-07-26 Thread Apache Jenkins Server
See

Re: Review Request 12433: Updated proc::status to handle race with checking if a file exists and opening the file.

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12433/#review24004 --- Ship it! Ship It! - Ben Mahler On July 10, 2013, 7:36 a.m., Benj

Re: Review Request 12435: Added proc::cmdline to get better command lines in os::Process.

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12435/#review24002 --- 3rdparty/libprocess/3rdparty/stout/include/stout/proc.hpp

Re: Review Request 12437: Updated Mesos to use the new os::killtree.

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12437/#review24000 --- This will need a rebase on Vinod's change to libprocess-ify the Comm

Re: Review Request 12436: Updated os::killtree:

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12436/#review23998 --- Cool! Looking forward for ProcessTree and a better os::killtree! 3

Re: Review Request 12887: Style /browse to distinguish folders

2013-07-26 Thread Ross Allen
> On July 26, 2013, 10:34 p.m., Ben Mahler wrote: > > src/webui/master/static/browse.html, lines 34-37 > > > > > > Nice! We should consider adding a ".." folder as well to all the > > listings if that makes it nicer

Re: Review Request 11124: Kill tasks that never properly launch.

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11124/#review23997 --- Ship it! Ship It! - Ben Mahler On July 25, 2013, 1:28 a.m., Bren

Re: Review Request 11130: Allowed some JVM memory vars to be configured.

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11130/#review23990 --- Ship it! hadoop/mapred-site.xml.patch

Re: Review Request 12922: Add running task count to state info.

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12922/#review23989 --- Ship it! Ship It! - Ben Mahler On July 24, 2013, 10:17 p.m., Bre

Re: Review Request 12887: Style /browse to distinguish folders

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12887/#review23987 --- Ship it! src/webui/master/static/browse.html

Re: Review Request 12880: Compute framework stats in one iteration

2013-07-26 Thread Ben Mahler
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12880/#review23985 --- Ship it! Ship It! - Ben Mahler On July 26, 2013, 6:23 p.m., Ross

Re: Review Request 12980: Update email addresses for the dev list.

2013-07-26 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12980/#review23979 --- Ship it! Awesome, thank you! Will get this submitted today. - Benj

Review Request 12980: Update email addresses for the dev list.

2013-07-26 Thread Matthew Silvey
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12980/ --- Review request for mesos. Bugs: MESOS-564 https://issues.apache.org/jira/br

[jira] [Created] (MESOS-590) Add Java and Python bindings for the Resources and Values classes

2013-07-26 Thread Thomas Marshall (JIRA)
Thomas Marshall created MESOS-590: - Summary: Add Java and Python bindings for the Resources and Values classes Key: MESOS-590 URL: https://issues.apache.org/jira/browse/MESOS-590 Project: Mesos

[jira] [Resolved] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vinod Kone resolved MESOS-587. -- Resolution: Fixed Assignee: Vinod Kone > MesosNativeLibrary.java doesn't read environment vari

[jira] [Commented] (MESOS-589) Add Mesos branding to web UI

2013-07-26 Thread Dave Lester (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-589?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13721093#comment-13721093 ] Dave Lester commented on MESOS-589: --- Done. This would be great! Additionally, if there ar

[jira] [Commented] (MESOS-589) Add Mesos branding to web UI

2013-07-26 Thread Benjamin Hindman (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-589?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13721083#comment-13721083 ] Benjamin Hindman commented on MESOS-589: I imagine [~davelester] can share the colo

Re: Review Request 12975: Fixed MesosNativeLibrary to use MESOS_NATIVE_LIBRARY environment variable instead of system property.

2013-07-26 Thread Benjamin Hindman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12975/#review23965 --- Ship it! Ship It! - Benjamin Hindman On July 26, 2013, 6:18 p.m.

Re: Review Request 12887: Style /browse to distinguish folders

2013-07-26 Thread Ross Allen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12887/ --- (Updated July 26, 2013, 7:03 p.m.) Review request for mesos. Changes ---

Re: Review Request 12887: Style /browse to distinguish folders

2013-07-26 Thread Ross Allen
> On July 25, 2013, 11:55 p.m., Ben Mahler wrote: > > src/webui/master/static/browse.html, lines 79-82 > > > > > > Does this still render well when there are no files? I added a "No files in this directory." message wh

Re: Review Request 12887: Style /browse to distinguish folders

2013-07-26 Thread Ross Allen
> On July 25, 2013, 9:05 p.m., Ben Mahler wrote: > > src/webui/master/static/browse.html, lines 42-43 > > > > > > Looks a bit odd that some are right aligned, and some are left aligned > > IMO, was this the cleanest lo

Re: Review Request 12887: Style /browse to distinguish folders

2013-07-26 Thread Ross Allen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12887/ --- (Updated July 26, 2013, 6:57 p.m.) Review request for mesos. Changes ---

[jira] [Created] (MESOS-589) Add Mesos branding to web UI

2013-07-26 Thread Ross Allen (JIRA)
Ross Allen created MESOS-589: Summary: Add Mesos branding to web UI Key: MESOS-589 URL: https://issues.apache.org/jira/browse/MESOS-589 Project: Mesos Issue Type: Improvement Components

[jira] [Commented] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13721038#comment-13721038 ] Vinod Kone commented on MESOS-587: -- Yay. Can't wait for more contributions!

[jira] [Commented] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Shingo Omura (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13721035#comment-13721035 ] Shingo Omura commented on MESOS-587: Thanks Vinod! It was so tiny patch, but I'm happy

Re: Review Request 12880: Compute framework stats in one iteration

2013-07-26 Thread Ross Allen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12880/ --- (Updated July 26, 2013, 6:23 p.m.) Review request for mesos. Changes ---

[jira] [Commented] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13721021#comment-13721021 ] Vinod Kone commented on MESOS-587: -- Thanks Shingo Omura for the patch! https://reviews.ap

[jira] [Updated] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Vinod Kone (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Vinod Kone updated MESOS-587: - Fix Version/s: 0.14.0 > MesosNativeLibrary.java doesn't read environment variable > "MESOS_NATIVE_LIB

Review Request 12975: Fixed MesosNativeLibrary to use MESOS_NATIVE_LIBRARY environment variable instead of system property.

2013-07-26 Thread Vinod Kone
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12975/ --- Review request for mesos, Benjamin Hindman and Ben Mahler. Bugs: MESOS-587

[jira] [Created] (MESOS-588) Add option to master to send stdout and stderr to syslog

2013-07-26 Thread Jason Dusek (JIRA)
Jason Dusek created MESOS-588: - Summary: Add option to master to send stdout and stderr to syslog Key: MESOS-588 URL: https://issues.apache.org/jira/browse/MESOS-588 Project: Mesos Issue Type: Ne

Re: Review Request 12834: Iterate frameworks successively to save concat

2013-07-26 Thread Ross Allen
> On July 25, 2013, 9:18 p.m., Ben Mahler wrote: > > Note that this would be removed once we make a change in the backend to > > remove from completedFrameworks when a framework re-registered. Feel free > > to file a ticket against me. > > > > The original bug is https://issues.apache.org/jira

Re: Review Request 12832: Iterate over all offers only once

2013-07-26 Thread Ross Allen
> On July 25, 2013, 9:13 p.m., Ben Mahler wrote: > > Thanks! > > > > I'm noticing angular.forEach and I'm not sure I find it as clean as _.each > > (see my comment on https://reviews.apache.org/r/12830/). > > Ross Allen wrote: > angular.forEach is equivalent to _.each but with a bit longer

Re: Review Request 12830: Prefer "angular.forEach" to Underscore's "_.each"

2013-07-26 Thread Ross Allen
> On July 25, 2013, 9 p.m., Ben Mahler wrote: > > I actually quite like the underscore.js library, what would be the benefit > > from eliminating it? > > Ross Allen wrote: > `forEach` is implemented in Angular, and so Underscore's implementation > is duplicate JavaScript. Underscore is gre

[jira] [Commented] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Benjamin Hindman (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13720941#comment-13720941 ] Benjamin Hindman commented on MESOS-587: Yes, this is definitely a bug! We use Revi

[jira] [Updated] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Shingo Omura (JIRA)
[ https://issues.apache.org/jira/browse/MESOS-587?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Shingo Omura updated MESOS-587: --- Attachment: 0001-change-getProperty-to-getenv-in-MesosNativeLibrary.j.patch patch. >

[jira] [Created] (MESOS-587) MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY".

2013-07-26 Thread Shingo Omura (JIRA)
Shingo Omura created MESOS-587: -- Summary: MesosNativeLibrary.java doesn't read environment variable "MESOS_NATIVE_LIBRARY". Key: MESOS-587 URL: https://issues.apache.org/jira/browse/MESOS-587 Project: Me