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

Harish Butani commented on HIVE-3981:
-------------------------------------

I attached the files. Several things:
- the rc & seq file were in the original patch. So why do you need them here
- We have struggled with this issue. The reason to keep all tests in 1 file is 
we make sure all the ptf tests run before every commit. Can we have a junit 
suite for all these tests. I would request that developers continue to make 
sure all these tests run before a commit.
- A hack we use to speed up running all the tests is to explicitly set 
'runningViaChild = false' line 133 MapRedTask.java. The 70 tests run in under 5 
minutes with this setting.
                
> Split up tests in ptf_general_queries.q
> ---------------------------------------
>
>                 Key: HIVE-3981
>                 URL: https://issues.apache.org/jira/browse/HIVE-3981
>             Project: Hive
>          Issue Type: Task
>          Components: PTF-Windowing
>            Reporter: Ashutosh Chauhan
>            Assignee: Ashutosh Chauhan
>         Attachments: hive-3981.patch, part.rc, part.seq
>
>
> ptf_general_queries.q has 62 queries currently and it takes nearly 20 minutes 
> on my laptop to finish. I think we should break it down in smaller .q files 
> otherwise adding a new query and debugging it will be a pain. I have split 
> out rcfile and seqfile tests from it to begin. Also, this test currently 
> fails because original patch didn't had .rc and .seq files (they were 
> binary). 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to