Github user JeffreyLyonsD2L commented on the pull request:
https://github.com/apache/phoenix/pull/104#issuecomment-128835914
Thanks for all the feedback @elilevine and @samarthjain! I'm east coast, so
I'm just heading out of the office now, but I should be able to get the changes
up sometime tomorrow.
The above question kind of goes outside my realm of interaction with
Phoenix so I'm not sure if my handle on it is correct. Is the point of the
TenantId in a table like System.Catalog to tie specific schema to a tenant? If
so I would lean toward leaving as VARCHAR since any TenantId has to be at least
convertible to a VARCHAR since it comes down the connection as one, though I
agree the your 'correct' solution seems like the most complete way forward.
---
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.
---
