phoenix
phoenix copied to clipboard
PHOENIX-6579 ACL check doesn't honor the namespace mapping for mapped…
… views.
Thanks for the submission. I think the changes look okay but can you please add a test to ensure this doesn't regress in the future?
retest
This got me thinking that we should in fact
use "^"+tableName.getNameWithNamespaceInclAsString()+"$"
to avoid a similar stituation, where we have permission for ns:prefixTable,
and we do not add the required new permission for ns:prefixTableFoo
because we THINK we already have it.
I think I'm going to open am HBase ticket about the horrific API design (if one doesn't exist yet).
Closing as this has been merged.