You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hive.apache.org by "Zoltan Haindrich (JIRA)" <ji...@apache.org> on 2017/10/10 21:04:00 UTC

[jira] [Commented] (HIVE-17758) NOTIFICATION_SEQUENCE_LOCK_RETRY_SLEEP_INTERVAL.defaultLongVal is -1

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

Zoltan Haindrich commented on HIVE-17758:
-----------------------------------------

[~anishek] could you please take a look?

> NOTIFICATION_SEQUENCE_LOCK_RETRY_SLEEP_INTERVAL.defaultLongVal is -1
> --------------------------------------------------------------------
>
>                 Key: HIVE-17758
>                 URL: https://issues.apache.org/jira/browse/HIVE-17758
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Zoltan Haindrich
>            Assignee: Zoltan Haindrich
>         Attachments: HIVE-17758.01.patch, HIVE-17758.02.patch
>
>
> HIVE-16886 introduced a retry logic; which has a configurable retry interval. unfortunately {{HiveConf}} has some public fields which at first glance seems to be usefull to pass as arguments to other methods - but in this case the default value is not even loaded into the field read by the code.. and because of that the innocent client code [here|https://github.com/apache/hive/blob/a974a9e6c4659f511e0b5edb97ce340a023a2e26/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java#L8554]  have used a {{-1}} value incorrectly which eventually caused an exception [here|https://github.com/apache/hive/blob/a974a9e6c4659f511e0b5edb97ce340a023a2e26/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java#L8581]:
> {code}
> 2017-10-10 11:22:37,638 ERROR [load-dynamic-partitions-12]: metastore.ObjectStore (ObjectStore.java:addNotificationEvent(7444)) - could not get lock for update
> java.lang.IllegalArgumentException: timeout value is negative
>         at java.lang.Thread.sleep(Native Method)
>         at org.apache.hadoop.hive.metastore.ObjectStore$RetryingExecutor.run(ObjectStore.java:7407)
>         at org.apache.hadoop.hive.metastore.ObjectStore.lockForUpdate(ObjectStore.java:7361)
>         at org.apache.hadoop.hive.metastore.ObjectStore.addNotificationEvent(ObjectStore.java:7424)
>         at sun.reflect.GeneratedMethodAccessor71.invoke(Unknown Source)
>         at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
>         at java.lang.reflect.Method.invoke(Method.java:498)
> [...]
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)