[jira] [Updated] (PHOENIX-7824) ConcurrentMutationsExtendedIT fix wrong TimeUnit on doneSignal.await

["Viraj Jasani (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/PHOENIX-7824?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Viraj Jasani updated PHOENIX-7824:
----------------------------------
    Fix Version/s: 5.3.2
                       (was: 5.3.1)

> ConcurrentMutationsExtendedIT fix wrong TimeUnit on doneSignal.await
> --------------------------------------------------------------------
>
>                 Key: PHOENIX-7824
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-7824
>             Project: Phoenix
>          Issue Type: Sub-task
>          Components: test
>            Reporter: Andrew Kyle Purtell
>            Assignee: Andrew Kyle Purtell
>            Priority: Minor
>             Fix For: 5.4.0, 5.3.2
>
>
> Two call sites in {{ConcurrentMutationsExtendedIT}} calls {{{}doneSignal.await(ROW_LOCK_WAIT_TIME + 5000, TimeUnit.SECONDS){}}}, but {{ROW_LOCK_WAIT_TIME}} is the HBase row-lock-wait constant in milliseconds (default 10000). Passing {{TimeUnit.SECONDS}} turned the intended 15-second wait into a 4+ hour timeout. Additionally, the return value of await was unchecked, allowing silent under-completion to bypass the verification phase and let the test "pass" by accident.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)