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

Szehon Ho commented on HIVE-7254:
---------------------------------

Hi Lefty thanks for looking at it.  The PTest framework is not a released 
product per se, its just a evolving framework used by devs always in latest 
stage, so I think we dont need to maintain old info as not sure anyone would 
ever use the old framework.

Thanks for finding all references to that page.  As I am looking through, I was 
thinking, one way to have less disruption is instead of deleting, to replace 
that page contents with what Gunther added (which works for both the normal 
build that dev's do locally, and the Ptest framework).  How to add a MiniMR 
test was never documented even in the past form and might be useful.  I guess 
either Gunther or I could take a stab at it.  

If so, the page (and thus the links) would still have to be renamed though from 
"MiniMR and PTest2" to just as now its a general case, should be "MiniCluster 
tests" or something of that nature.  And one parent reference should still be 
removed, namely the one from the PTest framework page: 
[https://cwiki.apache.org/confluence/display/Hive/Hive+PreCommit+Patch+Testing|https://cwiki.apache.org/confluence/display/Hive/Hive+PreCommit+Patch+Testing].
  Let me know what you think.

> Enhance Ptest framework config to auto-pick up list of MiniXXXDriver's test
> ---------------------------------------------------------------------------
>
>                 Key: HIVE-7254
>                 URL: https://issues.apache.org/jira/browse/HIVE-7254
>             Project: Hive
>          Issue Type: Test
>          Components: Testing Infrastructure
>            Reporter: Szehon Ho
>            Assignee: Szehon Ho
>         Attachments: trunk-mr2.properties
>
>
> Today, the Hive PTest infrastructure has a test-driver configuration called 
> "directory", so it will run all the qfiles under that directory for that 
> driver.  For example, CLIDriver is configured with directory 
> "ql/src/test/queries/clientpositive"
> However the configuration for the miniXXXDrivers (miniMRDriver, 
> miniMRDriverNegative, miniTezDriver) run only a select number of tests under 
> "directory".  So we have to use the "include" configuration to hard-code a 
> list of tests for it to run.  This is duplicating the list of each 
> miniDriver's tests already in the /itests/qtest pom file, and can get out of 
> date.
> It would be nice if both got their information the same way.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to