Hi all,

I was looking at two of Meghna's patch submissions (
https://reviews.apache.org/r/8355/  and
https://reviews.apache.org/r/8545 ) and realized that I'd like to have
an API guru take a look at the proposed modifications.

Review 8355 as 2 patches, one setup as the actual patch for review and
the followup fixes are included in an attachment.  You have to apply
both to see the expected result.

I've applied the patches and tested the compile + new unit tests, but
I'd like someone else to review the patches as well.  They effect
several command class files, modifying them to better support test
mock objects.  I don't want to push this commit without another pair
of eyes.

Thanks!

-chip

Reply via email to