You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@aurora.apache.org by Maxim Khutornenko <ma...@apache.org> on 2013/12/13 19:06:59 UTC

Review Request 16246: Fixing usage in /structdump

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

Review request for Aurora, Kevin Sweeney and Bill Farner.


Repository: aurora


Description
-------

Fixing usage in /structdump.


Diffs
-----

  src/main/java/com/twitter/aurora/scheduler/http/StructDump.java 8da8b801efc23219583d6c3c5c49761f9f9523d0 

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


Testing
-------

n/a


Thanks,

Maxim Khutornenko


Re: Review Request 16246: Fixing usage in /structdump

Posted by Bill Farner <wf...@apache.org>.

> On Dec. 13, 2013, 6:10 p.m., Bill Farner wrote:
> > Mind closing this in favor of https://reviews.apache.org/r/16062, and asking for the s/job/cron/ switch over there?
> 
> Maxim Khutornenko wrote:
>     Sure, did not notice that RB. Looks like it was never committed.

Yeah, it slipped through the cracks while we were figuring out why rbt was busted.


- Bill


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


On Dec. 13, 2013, 6:06 p.m., Maxim Khutornenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16246/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2013, 6:06 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Fixing usage in /structdump.
> 
> 
> Diffs
> -----
> 
>   src/main/java/com/twitter/aurora/scheduler/http/StructDump.java 8da8b801efc23219583d6c3c5c49761f9f9523d0 
> 
> Diff: https://reviews.apache.org/r/16246/diff/
> 
> 
> Testing
> -------
> 
> n/a
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>


Re: Review Request 16246: Fixing usage in /structdump

Posted by Maxim Khutornenko <ma...@apache.org>.

> On Dec. 13, 2013, 6:10 p.m., Bill Farner wrote:
> > Mind closing this in favor of https://reviews.apache.org/r/16062, and asking for the s/job/cron/ switch over there?

Sure, did not notice that RB. Looks like it was never committed.


- Maxim


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


On Dec. 13, 2013, 6:06 p.m., Maxim Khutornenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16246/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2013, 6:06 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Fixing usage in /structdump.
> 
> 
> Diffs
> -----
> 
>   src/main/java/com/twitter/aurora/scheduler/http/StructDump.java 8da8b801efc23219583d6c3c5c49761f9f9523d0 
> 
> Diff: https://reviews.apache.org/r/16246/diff/
> 
> 
> Testing
> -------
> 
> n/a
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>


Re: Review Request 16246: Fixing usage in /structdump

Posted by Bill Farner <wf...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/16246/#review30349
-----------------------------------------------------------


Mind closing this in favor of https://reviews.apache.org/r/16062, and asking for the s/job/cron/ switch over there?

- Bill Farner


On Dec. 13, 2013, 6:06 p.m., Maxim Khutornenko wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/16246/
> -----------------------------------------------------------
> 
> (Updated Dec. 13, 2013, 6:06 p.m.)
> 
> 
> Review request for Aurora, Kevin Sweeney and Bill Farner.
> 
> 
> Repository: aurora
> 
> 
> Description
> -------
> 
> Fixing usage in /structdump.
> 
> 
> Diffs
> -----
> 
>   src/main/java/com/twitter/aurora/scheduler/http/StructDump.java 8da8b801efc23219583d6c3c5c49761f9f9523d0 
> 
> Diff: https://reviews.apache.org/r/16246/diff/
> 
> 
> Testing
> -------
> 
> n/a
> 
> 
> Thanks,
> 
> Maxim Khutornenko
> 
>