HBase
  1. HBase
  2. HBASE-5845

Single Put should use RetriesExhaustedWithDetailsException in case any exception

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Won't Fix
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      In HBASE-5824, attempt was made to handle single Put execution separately. Put has two exception paths thereafter. It's better to keep one exception for easy exception handling.

        Issue Links

          Activity

          Jimmy Xiang created issue -
          Jimmy Xiang made changes -
          Field Original Value New Value
          Link This issue relates to HBASE-5824 [ HBASE-5824 ]
          Ted Yu made changes -
          Link This issue is related to HBASE-5824 [ HBASE-5824 ]
          Ted Yu made changes -
          Link This issue relates to HBASE-5824 [ HBASE-5824 ]
          Jimmy Xiang made changes -
          Description Due to change in HBASE-5824. Put has to exception paths now. It's better to stay the same for easy exception handling. Due to change in HBASE-5824. Put has two exception paths now. It's better to stay the same for easy exception handling.
          Ted Yu made changes -
          Description Due to change in HBASE-5824. Put has two exception paths now. It's better to stay the same for easy exception handling. In HBASE-5824, attempt was made to handle single Put execution separately. Put has two exception paths thereafter. It's better to keep one exception for easy exception handling.
          Hide
          Jimmy Xiang added a comment -

          To handle single Put execution separately, while still use the same RetriesExhaustedWithDetailsException,
          the code is kind of complex and it seems not give us much benefit. The reason for that is
          RetriesExhaustedWithDetailsException expects an action and the corresponding region server info.

          Let's re-open it later if it is really an issue.

          Show
          Jimmy Xiang added a comment - To handle single Put execution separately, while still use the same RetriesExhaustedWithDetailsException, the code is kind of complex and it seems not give us much benefit. The reason for that is RetriesExhaustedWithDetailsException expects an action and the corresponding region server info. Let's re-open it later if it is really an issue.
          Jimmy Xiang made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Won't Fix [ 2 ]
          Jimmy Xiang made changes -
          Link This issue is related to HBASE-6739 [ HBASE-6739 ]

            People

            • Assignee:
              Jimmy Xiang
              Reporter:
              Jimmy Xiang
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development