Uploaded image for project: 'Tajo'
  1. Tajo
  2. TAJO-984

Improve the default data type handling in RowStoreUtil

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.0
    • Component/s: Storage
    • Labels:
      None

      Description

      As discussed in TAJO-666, the current code for handling default data type has a potential bug which might incur unexpected operations. We need to improve this by throwing valid exceptions.

        Activity

        Hide
        hudson Hudson added a comment -

        FAILURE: Integrated in Tajo-master-build #340 (See https://builds.apache.org/job/Tajo-master-build/340/)
        TAJO-984: Improve the default data type handling in RowStoreUtil. (jihoon via hyunsik) (hyunsik: rev d3697d7633d6be9fd1f0d29182bf83dda11a2cab)

        • CHANGES
        • tajo-storage/src/main/java/org/apache/tajo/storage/RowStoreUtil.java
        Show
        hudson Hudson added a comment - FAILURE: Integrated in Tajo-master-build #340 (See https://builds.apache.org/job/Tajo-master-build/340/ ) TAJO-984 : Improve the default data type handling in RowStoreUtil. (jihoon via hyunsik) (hyunsik: rev d3697d7633d6be9fd1f0d29182bf83dda11a2cab) CHANGES tajo-storage/src/main/java/org/apache/tajo/storage/RowStoreUtil.java
        Hide
        hyunsik Hyunsik Choi added a comment -

        committed it to master branch. Thanks Jihoon!

        Show
        hyunsik Hyunsik Choi added a comment - committed it to master branch. Thanks Jihoon!
        Hide
        hyunsik Hyunsik Choi added a comment -

        +1

        The patch looks good to me. I've verified 'mvn clean install'.

        Show
        hyunsik Hyunsik Choi added a comment - +1 The patch looks good to me. I've verified 'mvn clean install'.

          People

          • Assignee:
            jihoonson Jihoon Son
            Reporter:
            jihoonson Jihoon Son
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development