counter_incs by incrementRows

[ummae]

related: #35

I am not sure that is proper way, but at least it works.

[landscape-bot]

Repository health increased by 0.50% when pulling d455db0 on ummae:master into 6205177 on wbolster:master.

• 1 new problem was found (including 0 errors and 1 code smell).
• 19 problems were fixed (including 2 errors and 5 code smells).

[wbolster]

Perhaps counters_inc is a better name? And can you move it below counter_dec for clarity?

[ummae]

Yes, I agree and I will move it below counter_dec.

[wbolster]

Hi, thanks for working on this. I've added a few questions/comments.

Any chance you can add some tests?

[ummae]

Sure, and I found some conventional problem on PEP8 styles(e.g. line break 79bytes, etc..).
I will re-PR after fixing it all.

[landscape-bot]

Repository health increased by 0.62% when pulling 209ce27 on ummae:master into 6205177 on wbolster:master.

• 1 new problem was found (including 0 errors and 1 code smell).
• 19 problems were fixed (including 2 errors and 5 code smells).

[landscape-bot]

Repository health increased by 0.67% when pulling c5761f6 on ummae:master into 6205177 on wbolster:master.

• 1 new problem was found (including 0 errors and 1 code smell).
• 19 problems were fixed (including 2 errors and 5 code smells).

[ummae]

@wbolster Hi would you review this commit?

[wbolster]

Thanks, tests look good.

[wbolster]

Thanks for working on this.

See comments about using a dict. Additionally, I think some mention of this in the user guide might make sense.

[landscape-bot]

Repository health increased by 0.67% when pulling 3ab00d2 on ummae:master into 6205177 on wbolster:master.

• 1 new problem was found (including 0 errors and 1 code smell).
• 19 problems were fixed (including 2 errors and 5 code smells).

[ummae]

I applied eventually ;)

[landscape-bot]

Repository health increased by 0.67% when pulling ff5c525 on ummae:master into 6205177 on wbolster:master.

• 1 new problem was found (including 0 errors and 1 code smell).
• 19 problems were fixed (including 2 errors and 5 code smells).

[ummae]

@wbolster Hi, I wonder do you mind to merge ;) and one question would you have plan happybase version2 for thrift2 protocol

[wbolster]

Hi, #96 also adds support for batch counter increments using a different API. I think the API proposed in #96 is friendlier for applications. Thoughts?

[philip-sterne]

Hi @wbolster, I'm happy to add tests for #96 if you prefer the different API? We tried to keep the api identical to the Batch idea.

[RajatGoyal]

Any progress on this, It would be great If i can increase counters in bulk.