Re: Namespaces review

2013-08-07 Thread Stack
On Tue, Aug 6, 2013 at 10:58 AM, Stack st...@duboce.net wrote: On Tue, Aug 6, 2013 at 10:38 AM, Enis Söztutar e...@hortonworks.comwrote: ... However, here is my +1 for the patch at HBASE-8408, with the condition that some outstanding comments are addressed and tests pass. I am confident

Namespaces review

2013-08-06 Thread Enis Söztutar
Hi devs, I have finished reviewing the 20-page patch for namespaces (HBASE-8015) and security for NSs(HBASE-8049) and wanted to drop a couple of quick notes. Here is an overview of the changes in case you are not following closely: - There is a new TableName class which encapsulates namespace

Re: Namespaces review

2013-08-06 Thread Stack
On Tue, Aug 6, 2013 at 10:38 AM, Enis Söztutar e...@hortonworks.com wrote: ... However, here is my +1 for the patch at HBASE-8408, with the condition that some outstanding comments are addressed and tests pass. I am confident that we can iron out all of the issues before 0.96. Thanks for