Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-03-21 Thread Jake Farrell
> On March 21, 2014, 2:52 p.m., Bill Farner wrote: > > Jake - do you agree with discarding this in favor of > > https://reviews.apache.org/r/17750/? Was waiting on Kevin to commit that and then was going to rebase the other changes that I did for this against his https://reviews.apache.org/r/1

Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-03-21 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17818/#review38101 --- Jake - do you agree with discarding this in favor of https://review

Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-02-11 Thread Bill Farner
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17818/#review34224 --- The move into src/test is still a no-go AFAICT, since we currently e

Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-02-06 Thread Jake Farrell
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17818/ --- (Updated Feb. 7, 2014, 1:45 a.m.) Review request for Aurora, Kevin Sweeney and

Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-02-06 Thread Jake Farrell
> On Feb. 6, 2014, 9:53 p.m., Kevin Sweeney wrote: > > No ship. AbstractCronIT is used by QuartzCronIT in > > https://reviews.apache.org/r/17750/ > > > > Merging AbstractCronIT and QuartzCronIT is on the list of followup TODOs > > for that review, so this will be mitigated then. AbstractCronI

Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-02-06 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17818/#review33891 --- I'll be out until Monday - I'll have another look then. - Kevin Swe

Re: Review Request 17818: AURORA-145:Test dependencies leak into distribution

2014-02-06 Thread Kevin Sweeney
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/17818/#review33846 --- No ship. AbstractCronIT is used by QuartzCronIT in https://reviews.