hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "stack (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (HBASE-8430) Cell decoder/scanner/etc. should not hide exceptions
Date Thu, 02 May 2013 19:58:16 GMT

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

stack updated HBASE-8430:
-------------------------

    Attachment: 8430.txt

Move HBaseIOException back into hbase-common and then have CellScanner#advance throw CellScannerAdvanceException,
a subclass of HBaseIOException.

Patch is big only because stuff is moved around.
                
> Cell decoder/scanner/etc. should not hide exceptions
> ----------------------------------------------------
>
>                 Key: HBASE-8430
>                 URL: https://issues.apache.org/jira/browse/HBASE-8430
>             Project: HBase
>          Issue Type: Bug
>          Components: IPC/RPC, Protobufs
>            Reporter: Sergey Shelukhin
>            Assignee: stack
>            Priority: Critical
>             Fix For: 0.95.1
>
>         Attachments: 8430.txt
>
>
> Cell scanner, base decoder, etc., hide IOException inside runtime exception. This can
lead to unexpected behavior because a lot of code only expects IOException. There's no logical
justification behind this hiding so it should be removed before it's too late (the sooner
we do it the less throws declarations need to be added)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Mime
View raw message