Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Switched type hints from HTable to HTableInterface #13

Merged
merged 1 commit into from Nov 27, 2012

Conversation

Projects
None yet
2 participants
Contributor

senior commented Nov 26, 2012

I found when upgrading to CDH4 (HBase version 0.92.1-cdh4.1.2), HTablePool is no longer returning an HTable instance, but an implementation of HTableInterface. HTable also implements this, so these changes should be backward compatible.

Owner

davidsantiago commented Nov 27, 2012

This looks fine to me, will apply.

Is this causing an actual error in your code, or preventive?

David

@davidsantiago davidsantiago merged commit 3843d53 into davidsantiago:develop Nov 27, 2012

Contributor

senior commented Nov 27, 2012

Actual error, ClassCastException I think. Was trying to see if some code works on the latest Cloudera distribution and hit it.

Owner

davidsantiago commented Nov 27, 2012

OK, thanks. I saw the potential for it to cause problems, so I put out 0.92.1 last night just in case. But it's good to know it's not just theoretical.

David

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment