Review Request 25108: Fix default logger in command runner.

2014-08-27 Thread Joshua Cohen
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25108/ --- Review request for Aurora and Brian Wickman. Bugs: AURORA-670

Re: Review Request 25108: Fix default logger in command runner.

2014-08-27 Thread Brian Wickman
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25108/#review51709 --- Ship it! Ship It! - Brian Wickman On Aug. 27, 2014, 6:40 p.m.,

Re: Review Request 25108: Fix default logger in command runner.

2014-08-27 Thread Brian Wickman
On Aug. 27, 2014, 10:32 p.m., Brian Wickman wrote: Ship It! For future reference, the protocol here now is the person providing final required ship-it (me, since I'm the only reviewer) merges the change to master if the review was published by a non-committer. I'll merge this promptly.

Re: Review Request 25108: Fix default logger in command runner.

2014-08-27 Thread Brian Wickman
On Aug. 27, 2014, 10:32 p.m., Brian Wickman wrote: Ship It! Brian Wickman wrote: For future reference, the protocol here now is the person providing final required ship-it (me, since I'm the only reviewer) merges the change to master if the review was published by a non-committer.

Re: Review Request 25108: Fix default logger in command runner.

2014-08-27 Thread Joshua Cohen
On Aug. 27, 2014, 10:32 p.m., Brian Wickman wrote: Ship It! Brian Wickman wrote: For future reference, the protocol here now is the person providing final required ship-it (me, since I'm the only reviewer) merges the change to master if the review was published by a non-committer.