The second ctor is only used by a mock, which I didn't look into much
so just added it to both the places.
I'll just make it final and inline it.

On Fri, Jun 6, 2014 at 5:28 PM, Chris Hostetter
<[email protected]> wrote:
>
> : @@ -119,6 +119,8 @@ public class CoreAdminHandler extends Re
>         ...
> : @@ -129,6 +131,8 @@ public class CoreAdminHandler extends Re
>
> Doing the same init in 2 places seems prone to future errors -- why not
> just inline it with the declaration of the "parallelExecutor" (and make it
> final while we're at it)
>
>
> -Hoss
> http://www.lucidworks.com/
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>



-- 

Anshum Gupta
http://www.anshumgupta.net

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to