Mahout
  1. Mahout
  2. MAHOUT-22

Several matrix exceptions are checked exceptions, but should be unchecked

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.1
    • Component/s: None
    • Labels:
      None

      Description

      IndexException and CardinalityException should be unchecked exceptions by analogy to ArrayIndexOutOfBoundsException

      Having them be checked causes lots of try/catches in code that doesn't have a chance of fixing the problem. It should be understood that these can happen in any matrix manipulation code just like IllegalArgumentException, ArithmeticException or ArrayIndexOutOfBoundsException can happen in any numerical codes.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        4d 17h 14m 1 Karl Wettin 04/Apr/08 18:33
        Resolved Resolved Closed Closed
        41s 1 Karl Wettin 04/Apr/08 18:34
        Grant Ingersoll made changes -
        Workflow jira [ 12427727 ] no-reopen-closed, patch-avail [ 12444678 ]
        Karl Wettin made changes -
        Fix Version/s 0.1 [ 12312976 ]
        Karl Wettin made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Karl Wettin added a comment -

        Committed revision 644784.

        Thanks Ted!

        Show
        Karl Wettin added a comment - Committed revision 644784. Thanks Ted!
        Karl Wettin made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Karl Wettin added a comment -

        Committed revision 644784.

        Thanks Ted!

        Show
        Karl Wettin added a comment - Committed revision 644784. Thanks Ted!
        Hide
        Karl Wettin added a comment -

        I'll commit this tomorrow given there are no more comments.

        Show
        Karl Wettin added a comment - I'll commit this tomorrow given there are no more comments.
        Karl Wettin made changes -
        Assignee Karl Wettin [ karl.wettin ]
        Ted Dunning made changes -
        Field Original Value New Value
        Attachment MAHOUT-22.patch [ 12378919 ]
        Hide
        Ted Dunning added a comment -

        Here are the trivial changes.

        Show
        Ted Dunning added a comment - Here are the trivial changes.
        Ted Dunning created issue -

          People

          • Assignee:
            Karl Wettin
            Reporter:
            Ted Dunning
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development