PHOENIX-5474 IndexTool should report the number of rows built

[kadirozde]

No description provided.

[gjacoby126]

Logic itself looks good -- could we have a test that verifies that the counters are incremented properly? (Might have to be two unit tests of the new Writable and the mapper since instrumenting MapReduce internals can be tough.)

[kadirozde]

Logic itself looks good -- could we have a test that verifies that the counters are incremented properly? (Might have to be two unit tests of the new Writable and the mapper since instrumenting MapReduce internals can be tough.)

@gjacoby126, I get the row count from the MR job and verify it in the new version of the PR. Would you please review it? Thanks

[gjacoby126]

+1, thanks @kadirozde (And very nice solution to getting the test counter.)

[kadirozde]

+1, thanks @kadirozde (And very nice solution to getting the test counter.)

Thank you for pointing out that we need to verify the counter reported by mappers.

[gjacoby126]

JIRA is resolved, closing PR.