Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38260/#review98508 --- Master (98f692d) is green with this patch. ./build-support/jenkins

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38260/#review98501 --- Ship it! Ship It! - Maxim Khutornenko On Sept. 10, 2015, 11:53 p

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38260/ --- (Updated Sept. 10, 2015, 4:53 p.m.) Review request for Aurora and Steve Niemitz

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Bill Farner
> On Sept. 10, 2015, 10:05 a.m., Zameer Manji wrote: > > src/main/python/apache/aurora/executor/bin/thermos_executor_main.py, line 43 > > > > > > I think this should go to stderr like logging and other diagnostic > >

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Zameer Manji
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38260/#review98422 --- Ship it! src/main/python/apache/aurora/executor/bin/thermos_execut

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38260/#review98416 --- Master (98f692d) is green with this patch. ./build-support/jenkins

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Bill Farner
> On Sept. 10, 2015, 9:36 a.m., Maxim Khutornenko wrote: > > src/main/python/apache/aurora/executor/bin/thermos_executor_main.py, line 43 > > > > > > why not using log instead? The log has not been set up at this poi

Re: Review Request 38260: Print a traceback when MesosExecutorDriver fails to load.

2015-09-10 Thread Maxim Khutornenko
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/38260/#review98411 --- src/main/python/apache/aurora/executor/bin/thermos_executor_main.py