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

ASF GitHub Bot commented on PHOENIX-2822:
-----------------------------------------

Github user churrodog commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/158#discussion_r58735016
  
    --- Diff: 
phoenix-core/src/it/java/org/apache/phoenix/end2end/AbsFunctionEnd2EndIT.java 
---
    @@ -43,9 +44,10 @@ public void initTable() throws Exception {
             Connection conn = null;
             PreparedStatement stmt = null;
             try {
    +            System.out.println("TABLE_NAME = " + TABLE_NAME);
    --- End diff --
    
    no problem


> Tests that extend BaseHBaseManagedTimeIT are very slow
> ------------------------------------------------------
>
>                 Key: PHOENIX-2822
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-2822
>             Project: Phoenix
>          Issue Type: Improvement
>    Affects Versions: 4.8.0
>            Reporter: churro morales
>            Assignee: churro morales
>              Labels: HBASEDEPENDENCIES
>
> Since I am trying to refactor out all the hbase private dependencies, I have 
> to constantly run tests to make sure I didn't break anything.  The tests that 
> extend BaseHBaseManagedTimeIT are very slow as they have to delete all 
> non-system tables after every test case.  This takes around 5-10 seconds to 
> accomplish.  This adds significant time to the test suite. 
> I created a new class named: BaseHBaseManagedTimeTableReuseIT and it creates 
> a random table name such that we dont have collisions for tests.  It also 
> doesn't do any cleanup after each test case or class because these table 
> names should be unique.  I moved about 30-35 tests out from 
> BaseHBaseManagedTimeIT to BaseHBaseManagedTimeTableReuseIT and it 
> significantly improved the overall time it takes to run tests.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to