sharmaar12 commented on PR #7149: URL: https://github.com/apache/hbase/pull/7149#issuecomment-3233852684
> This mostly LGTM so far. I left some comments. Also, I see this is still a draft, so there are probably more changes incoming anyway. > > One question I have is what if a table has the same name in two different namespaces? I see method signatures for `refreshHfiles()` that are: > > ``` > refreshHFiles(final TableName tableName) > refreshHFiles(final String namespace) > refreshHFiles() > ``` > > Will the user ever need something like the following? > > ``` > refreshHFiles(final String namespace, TableName tableName) > ``` Actually we are not allowing user to pass both namespace and tableName together as parameter. We will send exception in case user calls `refresh_hfiles "NAMESPACE" => "test_ns", "TABLE_NAME" => "test"` , this is handled in ruby script in newer commit. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
