Github user JeffreyLyonsD2L commented on the pull request:
https://github.com/apache/phoenix/pull/104#issuecomment-129597825
Hey @elilevine, thanks and I appreciate the review, I've been learning a
lot :)
In terms of our use case, we have a pre-existing schema that we don't want
to modify. For one, it would be a significant refactor to some of our other
code in order to handle converting our existing TenantIds to strings.
The other issue is that we have a lot of traffic that is interacting with
the cluster directly through HBase and not through Phoenix. For this reason we
like having the TenantIds as fixed length, so that the rowkeys are easy and
logical to parse.
It also seems reasonable to me that if the TenantId is naturally a number
instead of a name that it could be represented with one, instead of forcing a
conversion to string when we store it.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
