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

Ming Ma commented on YARN-1987:
-------------------------------

Jason,

1. LeveldbIterator.close rethrows IOException instead of DBException. Just 
wonder which is better, given JniDBFactory.factory.open throws DBException.
2. It seems "store open via JniDBFactory.factory.open" can also be useful to 
put into a wrapper class, to take care of "catch the exception if the store 
doesn't exist and create a new one". Perhaps that will be another jira.

> Wrapper for leveldb DBIterator to aid in handling database exceptions
> ---------------------------------------------------------------------
>
>                 Key: YARN-1987
>                 URL: https://issues.apache.org/jira/browse/YARN-1987
>             Project: Hadoop YARN
>          Issue Type: Improvement
>    Affects Versions: 2.4.0
>            Reporter: Jason Lowe
>            Assignee: Jason Lowe
>         Attachments: YARN-1987.patch
>
>
> Per discussions in YARN-1984 and MAPREDUCE-5652, it would be nice to have a 
> utility wrapper around leveldb's DBIterator to translate the raw 
> RuntimeExceptions it can throw into DBExceptions to make it easier to handle 
> database errors while iterating.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to