[ 
https://issues.apache.org/jira/browse/HBASE-5217?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13245537#comment-13245537
 ] 

Alex Newman commented on HBASE-5217:
------------------------------------

This <bold>is</bold> dependent on the other being committed first. Sorry about 
the spacing will fix when I am back from vegas.  I think the reason this patch 
had them consolidated was
  * Runs all of the tests under a single JUnit test method.  We
-   * consolidate all testing to one method because HBaseClusterTestCase
-   * is prone to OutOfMemoryExceptions when there are three or more
-   * JUnit test methods.
-   *
-   * @throws Exception

I think this is mostly fixed now though.
                
> Reenable the thrift tests, and add a new one for getRegionInfo
> --------------------------------------------------------------
>
>                 Key: HBASE-5217
>                 URL: https://issues.apache.org/jira/browse/HBASE-5217
>             Project: HBase
>          Issue Type: Improvement
>            Reporter: Alex Newman
>            Assignee: Alex Newman
>            Priority: Minor
>         Attachments: 0001-Fixing-thrift-tests-v2.patch, 
> 0001-Fixing-thrift-tests.patch, 
> 0002-HBASE-5217.-Reenable-the-thrift-tests-and-add-a-new-.patch, 
> zzzz-hbase-posix4e #92 Console [Jenkins].pdf
>
>
> At some point we disabled tests for the thrift server. In addition, it looks 
> like the getRegionInfo no longer functions. I'd like to reenable the tests 
> and add one for getRegionInfo. I had to write this to test my changes in 
> HBASE-2600 anyway. I figured I would break it out. We shouldn't commit it 
> until we have fixed getting the regioninfo from the thriftserver.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to