You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mesos.apache.org by Ian Downes <ia...@gmail.com> on 2014/04/02 19:55:42 UTC

Review Request 19952: Reduce stack size for threads in proc tests.

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/
-----------------------------------------------------------

Review request for mesos, Jie Yu and Vinod Kone.


Bugs: MESOS-1152
    https://issues.apache.org/jira/browse/MESOS-1152


Repository: mesos-git


Description
-------

See summary.


Diffs
-----

  3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 

Diff: https://reviews.apache.org/r/19952/diff/


Testing
-------

make check # Linux


Thanks,

Ian Downes


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Mesos ReviewBot <de...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39331
-----------------------------------------------------------


Patch looks great!

Reviews applied: [19952]

All tests passed.

- Mesos ReviewBot


On April 2, 2014, 5:55 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 5:55 p.m.)
> 
> 
> Review request for mesos, Jie Yu and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Benjamin Hindman <be...@berkeley.edu>.

> On April 3, 2014, 9:41 p.m., Vinod Kone wrote:
> > LGTM modulo using the new os::read() (once that gets submitted).

Just got submitted!


- Benjamin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39478
-----------------------------------------------------------


On April 2, 2014, 10:05 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 10:05 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39478
-----------------------------------------------------------

Ship it!


LGTM modulo using the new os::read() (once that gets submitted).

- Vinod Kone


On April 2, 2014, 10:05 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 10:05 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39488
-----------------------------------------------------------

Ship it!


Ship It!

- Vinod Kone


On April 3, 2014, 10:59 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 3, 2014, 10:59 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Ian Downes <ia...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/
-----------------------------------------------------------

(Updated April 3, 2014, 10:59 p.m.)


Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.


Changes
-------

Revised for updated os::read()


Bugs: MESOS-1152
    https://issues.apache.org/jira/browse/MESOS-1152


Repository: mesos-git


Description
-------

See summary.


Diffs (updated)
-----

  3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 

Diff: https://reviews.apache.org/r/19952/diff/


Testing
-------

make check # Linux


Thanks,

Ian Downes


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Mesos ReviewBot <de...@mesos.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39397
-----------------------------------------------------------


Bad patch!

Reviews applied: [19952]

Failed command: make check

Error:
 Making check in .
make[1]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot'
make[1]: Nothing to be done for `check-am'.
make[1]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot'
Making check in 3rdparty
make[1]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
make  check-recursive
make[2]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
Making check in libprocess
make[3]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess'
Making check in 3rdparty
make[4]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make  check-recursive
make[5]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
Making check in stout
make[6]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty/stout'
make[6]: Nothing to be done for `check'.
make[6]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty/stout'
make[6]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make  libgmock.la stout-tests
make[7]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[7]: `libgmock.la' is up to date.
make[7]: `stout-tests' is up to date.
make[7]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make  check-local
make[7]: Entering directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
./stout-tests
[==========] Running 118 tests from 24 test cases.
[----------] Global test environment set-up.
[----------] 3 tests from Stout
[ RUN      ] Stout.Bytes
[       OK ] Stout.Bytes (0 ms)
[ RUN      ] Stout.Set
[       OK ] Stout.Set (0 ms)
[ RUN      ] Stout.Some
[       OK ] Stout.Some (0 ms)
[----------] 3 tests from Stout (0 ms total)

[----------] 4 tests from Cache
[ RUN      ] Cache.Insert
[       OK ] Cache.Insert (0 ms)
[ RUN      ] Cache.Update
[       OK ] Cache.Update (0 ms)
[ RUN      ] Cache.Erase
[       OK ] Cache.Erase (0 ms)
[ RUN      ] Cache.LRUEviction
[       OK ] Cache.LRUEviction (0 ms)
[----------] 4 tests from Cache (0 ms total)

[----------] 4 tests from DurationTest
[ RUN      ] DurationTest.Comparison
[       OK ] DurationTest.Comparison (0 ms)
[ RUN      ] DurationTest.ParseAndTry
[       OK ] DurationTest.ParseAndTry (0 ms)
[ RUN      ] DurationTest.Arithmetic
[       OK ] DurationTest.Arithmetic (0 ms)
[ RUN      ] DurationTest.OutputFormat
[       OK ] DurationTest.OutputFormat (0 ms)
[----------] 4 tests from DurationTest (0 ms total)

[----------] 1 test from ErrorTest
[ RUN      ] ErrorTest.Test
[       OK ] ErrorTest.Test (0 ms)
[----------] 1 test from ErrorTest (0 ms total)

[----------] 12 tests from FlagsTest
[ RUN      ] FlagsTest.Load
[       OK ] FlagsTest.Load (5 ms)
[ RUN      ] FlagsTest.Add
[       OK ] FlagsTest.Add (0 ms)
[ RUN      ] FlagsTest.Flags
[       OK ] FlagsTest.Flags (0 ms)
[ RUN      ] FlagsTest.LoadFromEnvironment
[       OK ] FlagsTest.LoadFromEnvironment (0 ms)
[ RUN      ] FlagsTest.LoadFromCommandLine
[       OK ] FlagsTest.LoadFromCommandLine (0 ms)
[ RUN      ] FlagsTest.LoadFromCommandLineWithNonFlags
[       OK ] FlagsTest.LoadFromCommandLineWithNonFlags (0 ms)
[ RUN      ] FlagsTest.Stringification
[       OK ] FlagsTest.Stringification (0 ms)
[ RUN      ] FlagsTest.DuplicatesFromEnvironment
[       OK ] FlagsTest.DuplicatesFromEnvironment (0 ms)
[ RUN      ] FlagsTest.DuplicatesFromCommandLine
[       OK ] FlagsTest.DuplicatesFromCommandLine (0 ms)
[ RUN      ] FlagsTest.Errors
[       OK ] FlagsTest.Errors (0 ms)
[ RUN      ] FlagsTest.Usage
[       OK ] FlagsTest.Usage (0 ms)
[ RUN      ] FlagsTest.Duration
[       OK ] FlagsTest.Duration (0 ms)
[----------] 12 tests from FlagsTest (5 ms total)

[----------] 1 test from GzipTest
[ RUN      ] GzipTest.CompressDecompressString
[       OK ] GzipTest.CompressDecompressString (95 ms)
[----------] 1 test from GzipTest (95 ms total)

[----------] 2 tests from HashMapTest
[ RUN      ] HashMapTest.Insert
[       OK ] HashMapTest.Insert (0 ms)
[ RUN      ] HashMapTest.Contains
[       OK ] HashMapTest.Contains (0 ms)
[----------] 2 tests from HashMapTest (0 ms total)

[----------] 2 tests from HashsetTest
[ RUN      ] HashsetTest.Insert
[       OK ] HashsetTest.Insert (0 ms)
[ RUN      ] HashsetTest.Union
[       OK ] HashsetTest.Union (0 ms)
[----------] 2 tests from HashsetTest (0 ms total)

[----------] 8 tests from IntervalTest
[ RUN      ] IntervalTest.Interval
[       OK ] IntervalTest.Interval (0 ms)
[ RUN      ] IntervalTest.EmptyInterval
[       OK ] IntervalTest.EmptyInterval (0 ms)
[ RUN      ] IntervalTest.Constructor
[       OK ] IntervalTest.Constructor (0 ms)
[ RUN      ] IntervalTest.Addition
[       OK ] IntervalTest.Addition (0 ms)
[ RUN      ] IntervalTest.Subtraction
[       OK ] IntervalTest.Subtraction (0 ms)
[ RUN      ] IntervalTest.Intersection
[       OK ] IntervalTest.Intersection (0 ms)
[ RUN      ] IntervalTest.LargeInterval
[       OK ] IntervalTest.LargeInterval (0 ms)
[ RUN      ] IntervalTest.IntervalIteration
[       OK ] IntervalTest.IntervalIteration (0 ms)
[----------] 8 tests from IntervalTest (0 ms total)

[----------] 8 tests from JsonTest
[ RUN      ] JsonTest.DefaultValueIsNull
[       OK ] JsonTest.DefaultValueIsNull (1 ms)
[ RUN      ] JsonTest.BinaryData
[       OK ] JsonTest.BinaryData (0 ms)
[ RUN      ] JsonTest.NumberFormat
[       OK ] JsonTest.NumberFormat (0 ms)
[ RUN      ] JsonTest.BooleanFormat
[       OK ] JsonTest.BooleanFormat (0 ms)
[ RUN      ] JsonTest.BooleanAssignement
[       OK ] JsonTest.BooleanAssignement (0 ms)
[ RUN      ] JsonTest.CStringAssignment
[       OK ] JsonTest.CStringAssignment (0 ms)
[ RUN      ] JsonTest.NumericAssignment
[       OK ] JsonTest.NumericAssignment (0 ms)
[ RUN      ] JsonTest.parse
[       OK ] JsonTest.parse (0 ms)
[----------] 8 tests from JsonTest (1 ms total)

[----------] 5 tests from LinkedHashmapTest
[ RUN      ] LinkedHashmapTest.Put
[       OK ] LinkedHashmapTest.Put (0 ms)
[ RUN      ] LinkedHashmapTest.Contains
[       OK ] LinkedHashmapTest.Contains (0 ms)
[ RUN      ] LinkedHashmapTest.Erase
[       OK ] LinkedHashmapTest.Erase (0 ms)
[ RUN      ] LinkedHashmapTest.Keys
[       OK ] LinkedHashmapTest.Keys (0 ms)
[ RUN      ] LinkedHashmapTest.Values
[       OK ] LinkedHashmapTest.Values (0 ms)
[----------] 5 tests from LinkedHashmapTest (0 ms total)

[----------] 6 tests from MultimapTest/0, where TypeParam = Multimap<std::string, unsigned short>
[ RUN      ] MultimapTest/0.Put
[       OK ] MultimapTest/0.Put (0 ms)
[ RUN      ] MultimapTest/0.Remove
[       OK ] MultimapTest/0.Remove (0 ms)
[ RUN      ] MultimapTest/0.Size
[       OK ] MultimapTest/0.Size (0 ms)
[ RUN      ] MultimapTest/0.Keys
[       OK ] MultimapTest/0.Keys (0 ms)
[ RUN      ] MultimapTest/0.Iterator
[       OK ] MultimapTest/0.Iterator (0 ms)
[ RUN      ] MultimapTest/0.Foreach
[       OK ] MultimapTest/0.Foreach (0 ms)
[----------] 6 tests from MultimapTest/0 (0 ms total)

[----------] 6 tests from MultimapTest/1, where TypeParam = multihashmap<std::string, unsigned short>
[ RUN      ] MultimapTest/1.Put
[       OK ] MultimapTest/1.Put (0 ms)
[ RUN      ] MultimapTest/1.Remove
[       OK ] MultimapTest/1.Remove (0 ms)
[ RUN      ] MultimapTest/1.Size
[       OK ] MultimapTest/1.Size (0 ms)
[ RUN      ] MultimapTest/1.Keys
[       OK ] MultimapTest/1.Keys (0 ms)
[ RUN      ] MultimapTest/1.Iterator
[       OK ] MultimapTest/1.Iterator (0 ms)
[ RUN      ] MultimapTest/1.Foreach
[       OK ] MultimapTest/1.Foreach (0 ms)
[----------] 6 tests from MultimapTest/1 (0 ms total)

[----------] 2 tests from NetTest
[ RUN      ] NetTest.mac
[       OK ] NetTest.mac (1 ms)
[ RUN      ] NetTest.ip
[       OK ] NetTest.ip (1 ms)
[----------] 2 tests from NetTest (2 ms total)

[----------] 1 test from NoneTest
[ RUN      ] NoneTest.Test
[       OK ] NoneTest.Test (0 ms)
[----------] 1 test from NoneTest (0 ms total)

[----------] 3 tests from OptionTest
[ RUN      ] OptionTest.Min
[       OK ] OptionTest.Min (0 ms)
[ RUN      ] OptionTest.Max
[       OK ] OptionTest.Max (0 ms)
[ RUN      ] OptionTest.Comparison
[       OK ] OptionTest.Comparison (0 ms)
[----------] 3 tests from OptionTest (0 ms total)

[----------] 17 tests from OsTest
[ RUN      ] OsTest.environment
[       OK ] OsTest.environment (0 ms)
[ RUN      ] OsTest.rmdir
[       OK ] OsTest.rmdir (1 ms)
[ RUN      ] OsTest.nonblock
[       OK ] OsTest.nonblock (5 ms)
[ RUN      ] OsTest.touch
[       OK ] OsTest.touch (1 ms)
[ RUN      ] OsTest.readWriteString
[       OK ] OsTest.readWriteString (1 ms)
[ RUN      ] OsTest.find
[       OK ] OsTest.find (1 ms)
[ RUN      ] OsTest.bootId
[       OK ] OsTest.bootId (1 ms)
[ RUN      ] OsTest.uname
[       OK ] OsTest.uname (0 ms)
[ RUN      ] OsTest.sysname
[       OK ] OsTest.sysname (0 ms)
[ RUN      ] OsTest.release
[       OK ] OsTest.release (0 ms)
[ RUN      ] OsTest.sleep
[       OK ] OsTest.sleep (10 ms)
[ RUN      ] OsTest.pids
[       OK ] OsTest.pids (34 ms)
[ RUN      ] OsTest.children
[       OK ] OsTest.children (54 ms)
[ RUN      ] OsTest.process
[       OK ] OsTest.process (0 ms)
[ RUN      ] OsTest.processes
[       OK ] OsTest.processes (18 ms)
[ RUN      ] OsTest.killtree
[       OK ] OsTest.killtree (115 ms)
[ RUN      ] OsTest.pstree
[       OK ] OsTest.pstree (35 ms)
[----------] 17 tests from OsTest (277 ms total)

[----------] 1 test from ProtobufTest
[ RUN      ] ProtobufTest.JSON
[       OK ] ProtobufTest.JSON (1 ms)
[----------] 1 test from ProtobufTest (1 ms total)

[----------] 1 test from OsSendfileTest
[ RUN      ] OsSendfileTest.sendfile
[       OK ] OsSendfileTest.sendfile (0 ms)
[----------] 1 test from OsSendfileTest (0 ms total)

[----------] 1 test from OsSignalsTest
[ RUN      ] OsSignalsTest.suppress
[       OK ] OsSignalsTest.suppress (0 ms)
[----------] 1 test from OsSignalsTest (0 ms total)

[----------] 22 tests from StringsTest
[ RUN      ] StringsTest.Format
[       OK ] StringsTest.Format (0 ms)
[ RUN      ] StringsTest.Remove
[       OK ] StringsTest.Remove (0 ms)
[ RUN      ] StringsTest.Replace
[       OK ] StringsTest.Replace (0 ms)
[ RUN      ] StringsTest.Trim
[       OK ] StringsTest.Trim (0 ms)
[ RUN      ] StringsTest.Tokenize
[       OK ] StringsTest.Tokenize (0 ms)
[ RUN      ] StringsTest.TokenizeStringWithDelimsAtStart
[       OK ] StringsTest.TokenizeStringWithDelimsAtStart (0 ms)
[ RUN      ] StringsTest.TokenizeStringWithDelimsAtEnd
[       OK ] StringsTest.TokenizeStringWithDelimsAtEnd (0 ms)
[ RUN      ] StringsTest.TokenizeStringWithDelimsAtStartAndEnd
[       OK ] StringsTest.TokenizeStringWithDelimsAtStartAndEnd (0 ms)
[ RUN      ] StringsTest.TokenizeWithMultipleDelims
[       OK ] StringsTest.TokenizeWithMultipleDelims (0 ms)
[ RUN      ] StringsTest.TokenizeEmptyString
[       OK ] StringsTest.TokenizeEmptyString (0 ms)
[ RUN      ] StringsTest.TokenizeDelimOnlyString
[       OK ] StringsTest.TokenizeDelimOnlyString (0 ms)
[ RUN      ] StringsTest.TokenizeNullByteDelim
[       OK ] StringsTest.TokenizeNullByteDelim (0 ms)
[ RUN      ] StringsTest.SplitEmptyString
[       OK ] StringsTest.SplitEmptyString (0 ms)
[ RUN      ] StringsTest.SplitDelimOnlyString
[       OK ] StringsTest.SplitDelimOnlyString (0 ms)
[ RUN      ] StringsTest.Split
[       OK ] StringsTest.Split (0 ms)
[ RUN      ] StringsTest.SplitStringWithDelimsAtStart
[       OK ] StringsTest.SplitStringWithDelimsAtStart (0 ms)
[ RUN      ] StringsTest.SplitStringWithDelimsAtEnd
[       OK ] StringsTest.SplitStringWithDelimsAtEnd (0 ms)
[ RUN      ] StringsTest.SplitStringWithDelimsAtStartAndEnd
[       OK ] StringsTest.SplitStringWithDelimsAtStartAndEnd (0 ms)
[ RUN      ] StringsTest.SplitWithMultipleDelims
[       OK ] StringsTest.SplitWithMultipleDelims (0 ms)
[ RUN      ] StringsTest.Pairs
[       OK ] StringsTest.Pairs (0 ms)
[ RUN      ] StringsTest.StartsWith
[       OK ] StringsTest.StartsWith (0 ms)
[ RUN      ] StringsTest.Contains
[       OK ] StringsTest.Contains (0 ms)
[----------] 22 tests from StringsTest (0 ms total)

[----------] 1 test from Thread
[ RUN      ] Thread.local
[       OK ] Thread.local (0 ms)
[----------] 1 test from Thread (0 ms total)

[----------] 1 test from UUIDTest
[ RUN      ] UUIDTest.test
[       OK ] UUIDTest.test (0 ms)
[----------] 1 test from UUIDTest (0 ms total)

[----------] 6 tests from ProcTest
[ RUN      ] ProcTest.pids
[       OK ] ProcTest.pids (1 ms)
[ RUN      ] ProcTest.cpus
[       OK ] ProcTest.cpus (1 ms)
[ RUN      ] ProcTest.SystemStatus
[       OK ] ProcTest.SystemStatus (1 ms)
[ RUN      ] ProcTest.ProcessStatus
[       OK ] ProcTest.ProcessStatus (0 ms)
[ RUN      ] ProcTest.SingleThread
[       OK ] ProcTest.SingleThread (0 ms)
[ RUN      ] ProcTest.MultipleThreads
stout/tests/proc_tests.cpp:173: Failure
Value of: len
  Actual: 0
Expected: sizeof(size)
Which is: 8
[  FAILED  ] ProcTest.MultipleThreads (175 ms)
[----------] 6 tests from ProcTest (178 ms total)

[----------] Global test environment tear-down
[==========] 118 tests from 24 test cases ran. (560 ms total)
[  PASSED  ] 117 tests.
[  FAILED  ] 1 test, listed below:
[  FAILED  ] ProcTest.MultipleThreads

 1 FAILED TEST
make[7]: *** [check-local] Error 1
make[7]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[6]: *** [check-am] Error 2
make[6]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[5]: *** [check-recursive] Error 1
make[5]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[4]: *** [check] Error 2
make[4]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess/3rdparty'
make[3]: *** [check-recursive] Error 1
make[3]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty/libprocess'
make[2]: *** [check-recursive] Error 1
make[2]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
make[1]: *** [check] Error 2
make[1]: Leaving directory `/x1/jenkins/jenkins-slave/workspace/mesos-reviewbot/3rdparty'
make: *** [check-recursive] Error 1


- Mesos ReviewBot


On April 2, 2014, 10:05 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 10:05 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Ian Downes <ia...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/
-----------------------------------------------------------

(Updated April 2, 2014, 10:05 p.m.)


Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.


Changes
-------

Added BenH.


Bugs: MESOS-1152
    https://issues.apache.org/jira/browse/MESOS-1152


Repository: mesos-git


Description
-------

See summary.


Diffs
-----

  3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 

Diff: https://reviews.apache.org/r/19952/diff/


Testing
-------

make check # Linux


Thanks,

Ian Downes


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Dominic Hamon <dh...@twopensource.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39365
-----------------------------------------------------------



3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp
<https://reviews.apache.org/r/19952/#comment71744>

    'thread' might cause you problems in c++11. consider threadId


- Dominic Hamon


On April 2, 2014, 2:39 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 2:39 p.m.)
> 
> 
> Review request for mesos, Benjamin Hindman, Jie Yu, and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Ian Downes <ia...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/
-----------------------------------------------------------

(Updated April 2, 2014, 9:39 p.m.)


Review request for mesos, Jie Yu and Vinod Kone.


Changes
-------

Cleaned up sending thread ids from child to test.


Bugs: MESOS-1152
    https://issues.apache.org/jira/browse/MESOS-1152


Repository: mesos-git


Description
-------

See summary.


Diffs (updated)
-----

  3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 

Diff: https://reviews.apache.org/r/19952/diff/


Testing
-------

make check # Linux


Thanks,

Ian Downes


Re: Review Request 19952: Reduce stack size for threads in proc tests.

Posted by Vinod Kone <vi...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/19952/#review39334
-----------------------------------------------------------



3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp
<https://reviews.apache.org/r/19952/#comment71685>

    Also, you can kill this :)


- Vinod Kone


On April 2, 2014, 5:55 p.m., Ian Downes wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/19952/
> -----------------------------------------------------------
> 
> (Updated April 2, 2014, 5:55 p.m.)
> 
> 
> Review request for mesos, Jie Yu and Vinod Kone.
> 
> 
> Bugs: MESOS-1152
>     https://issues.apache.org/jira/browse/MESOS-1152
> 
> 
> Repository: mesos-git
> 
> 
> Description
> -------
> 
> See summary.
> 
> 
> Diffs
> -----
> 
>   3rdparty/libprocess/3rdparty/stout/tests/proc_tests.cpp cb2b6007928eef4b9b1f488b630d96565624185e 
> 
> Diff: https://reviews.apache.org/r/19952/diff/
> 
> 
> Testing
> -------
> 
> make check # Linux
> 
> 
> Thanks,
> 
> Ian Downes
> 
>