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

Hive QA commented on HIVE-22750:
--------------------------------



Here are the results of testing the latest attachment:
https://issues.apache.org/jira/secure/attachment/12999060/HIVE-22750.9.patch

{color:green}SUCCESS:{color} +1 due to 4 test(s) being added or modified.

{color:red}ERROR:{color} -1 due to 1 failed/errored test(s), 18171 tests 
executed
*Failed tests:*
{noformat}
org.apache.hadoop.hive.cli.TestMiniLlapLocalCliDriver.testCliDriver[schq_ingest]
 (batchId=186)
{noformat}

Test results: 
https://builds.apache.org/job/PreCommit-HIVE-Build/21479/testReport
Console output: https://builds.apache.org/job/PreCommit-HIVE-Build/21479/console
Test logs: http://104.198.109.242/logs/PreCommit-HIVE-Build-21479/

Messages:
{noformat}
Executing org.apache.hive.ptest.execution.TestCheckPhase
Executing org.apache.hive.ptest.execution.PrepPhase
Executing org.apache.hive.ptest.execution.YetusPhase
Executing org.apache.hive.ptest.execution.ExecutionPhase
Executing org.apache.hive.ptest.execution.ReportingPhase
Tests exited with: TestsFailedException: 1 tests failed
{noformat}

This message is automatically generated.

ATTACHMENT ID: 12999060 - PreCommit-HIVE-Build

> Consolidate LockType naming
> ---------------------------
>
>                 Key: HIVE-22750
>                 URL: https://issues.apache.org/jira/browse/HIVE-22750
>             Project: Hive
>          Issue Type: Improvement
>          Components: Transactions
>            Reporter: Zoltan Chovan
>            Assignee: Marton Bod
>            Priority: Minor
>         Attachments: HIVE-22750.1.patch, HIVE-22750.2.patch, 
> HIVE-22750.3.patch, HIVE-22750.4.patch, HIVE-22750.5.patch, 
> HIVE-22750.5.patch, HIVE-22750.6.patch, HIVE-22750.7.patch, 
> HIVE-22750.8.patch, HIVE-22750.9.patch
>
>
> Extend enum with string literal to remove unnecessary `id` to `char` casting 
> for the LockType:
> {code:java}
> switch (lockType) {
>             case EXCLUSIVE:
>               lockChar = LOCK_EXCLUSIVE;
>               break;
>             case SHARED_READ:
>               lockChar = LOCK_SHARED;
>               break;
>             case SHARED_WRITE:
>               lockChar = LOCK_SEMI_SHARED;
>               break;
>           }
> {code}
> Consolidate LockType naming in code and schema upgrade scripts:
> {code:java}
> CASE WHEN HL.`HL_LOCK_TYPE` = 'e' THEN 'exclusive' WHEN HL.`HL_LOCK_TYPE` = 
> 'r' THEN 'shared' WHEN HL.`HL_LOCK_TYPE` = 'w' THEN *'semi-shared'* END AS 
> LOCK_TYPE,
> {code}
> +*Lock types:*+
> EXCLUSIVE
>  EXCL_WRITE
>  SHARED_WRITE
>  SHARED_READ



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to