hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ted Yu (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HIVE-8343) Return value from BlockingQueue.offer() is not checked in DynamicPartitionPruner
Date Fri, 26 Jun 2015 22:12:05 GMT

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

Ted Yu updated HIVE-8343:
-------------------------
    Description: 
In addEvent() and processVertex(), there is call such as the following:

{code}
          queue.offer(event);
{code}
The return value should be checked. If false is returned, event would not have been queued.

Take a look at line 328 in:
http://fuseyism.com/classpath/doc/java/util/concurrent/LinkedBlockingQueue-source.html

  was:
In addEvent() and processVertex(), there is call such as the following:
{code}
          queue.offer(event);
{code}
The return value should be checked. If false is returned, event would not have been queued.

Take a look at line 328 in:
http://fuseyism.com/classpath/doc/java/util/concurrent/LinkedBlockingQueue-source.html


> Return value from BlockingQueue.offer() is not checked in DynamicPartitionPruner
> --------------------------------------------------------------------------------
>
>                 Key: HIVE-8343
>                 URL: https://issues.apache.org/jira/browse/HIVE-8343
>             Project: Hive
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Assignee: JongWon Park
>            Priority: Minor
>         Attachments: HIVE-8343.patch
>
>
> In addEvent() and processVertex(), there is call such as the following:
> {code}
>           queue.offer(event);
> {code}
> The return value should be checked. If false is returned, event would not have been queued.
> Take a look at line 328 in:
> http://fuseyism.com/classpath/doc/java/util/concurrent/LinkedBlockingQueue-source.html



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

Mime
View raw message