Github user JamesRTaylor commented on the pull request:
https://github.com/apache/phoenix/pull/135#issuecomment-160852616
Couple of other comments:
- You'll need to add code during installation of 4.7.0 (in
ConnectionQueryServicesImpl.init()) that takes care of removing old
coprocessors, adding new coprocessors, disabling local indexes and potentially
automatically kicking off a new index build
- We'll likely want to add a check that disallows column family names
prefixed with "C#" to prevent inadvertently treating these as local index
shadow column families. If we can get rid of the checks that use this prefix,
then we might not need to do this.
- Also, there's an edge case of an existing column family using the "C#"
prefix. Unlikely, but not sure how we'd want to handle this. Maybe error on an
attempt to create a local index on such a table?
---
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.
---
