[jira] [Commented] (PIG-5049) Cleanup e2e tests turing_jython.conf

["Rohini Palaniswamy (JIRA)" <ji...@apache.org>]

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

Rohini Palaniswamy commented on PIG-5049:
-----------------------------------------

1) Could you also move Jython_Command_1 to hcat.conf? Always have a failure on that one when I don't have hcat setup.
2) Jython_Error_7 
      - results = P.bind().run() does not have out1 and out2 bound. It will fail there instead of failing on iter = results[0].result("E").iterator(). 
      -  result.isSuccessful() should be results[0].isSuccessful(). It is not going to be executed as iterator call will fail. But still for sake of correctness.

> Cleanup e2e tests turing_jython.conf
> ------------------------------------
>
>                 Key: PIG-5049
>                 URL: https://issues.apache.org/jira/browse/PIG-5049
>             Project: Pig
>          Issue Type: Bug
>          Components: e2e harness
>            Reporter: Daniel Dai
>            Assignee: Daniel Dai
>             Fix For: 0.17.0
>
>         Attachments: PIG-5049-1.patch
>
>
> turing_jython is quite messy. In this ticket, I clean up the test suits and fix several issues:
> 1. The simple load/store with no param test repeated 3 times in different context
> 2. bring back Jython_Properties
> 3. Jython_Error_[4,5,7] are not test the right negative condition
> 4. comments does not match test



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