hive-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From anishek <...@git.apache.org>
Subject [GitHub] hive pull request #136: HIVE-15711 : Flaky TestEmbeddedThriftBinaryCLIServic...
Date Tue, 24 Jan 2017 08:28:26 GMT
GitHub user anishek opened a pull request:

    https://github.com/apache/hive/pull/136

    HIVE-15711 :  Flaky TestEmbeddedThriftBinaryCLIService.testTaskStatus

    The 'RUNNING' state can be set way before the job handle from MR frameworks is available
to be set on the server in the taskDisplay, hence there is a possibility that we can receive
jobStatus messages without the job id. Hence its not guaranteed  that it is  not null in running
status hence removing this condition check. This will be checked anyways when the status is
'FINISHED'.Printing the status before validation so in case of validation failures we will
know why it failed

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/anishek/hive HIVE-15711

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/hive/pull/136.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #136
    
----
commit 5d21cb9298fb62d50a0fe5defa2d2d5c88be23b8
Author: Anishek Agarwal <anishek@gmail.com>
Date:   2017-01-24T08:12:56Z

    HIVE-15711 :  Flaky TestEmbeddedThriftBinaryCLIService.testTaskStatus
    
    The 'RUNNING' state can be set way before the job handle from MR frameworks is available
to be set on the server in the taskDisplay, hence there is a possibility that we can receive
jobStatus messages without the job id. Hence its not guaranteed  that it is  not null in running
status hence removing this condition check. This will be checked anyways when the status is
'FINISHED'.Printing the status before validation so in case of validation failures we will
know why it failed

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message