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

Upgrade Bytes class and move some methods to others

    Details

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

      Description

      We borrowed Bytes class from HBase. We also added some utility methods to Bytes class.

      The main objective of this issue is to upgrade Bytes class and moved the methods we added to proper classes. This refactoring will make the future update of Bytes class easier.

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user hyunsik opened a pull request:

        https://github.com/apache/tajo/pull/86

        TAJO-965: Upgrade Bytes class and move some methods to others.

        You can merge this pull request into a Git repository by running:

        $ git pull https://github.com/hyunsik/tajo TAJO-965

        Alternatively you can review and apply these changes as the patch at:

        https://github.com/apache/tajo/pull/86.patch

        To close this pull request, make a commit to your master/trunk branch
        with (at least) the following in the commit message:

        This closes #86


        commit 78b2a42d807c7f244ecab814d6a99513a01daf33
        Author: Hyunsik Choi <hyunsik@apache.org>
        Date: 2014-07-21T07:03:12Z

        TAJO-965: Upgrade Bytes class and move some methods to others.


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user hyunsik opened a pull request: https://github.com/apache/tajo/pull/86 TAJO-965 : Upgrade Bytes class and move some methods to others. You can merge this pull request into a Git repository by running: $ git pull https://github.com/hyunsik/tajo TAJO-965 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/86.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #86 commit 78b2a42d807c7f244ecab814d6a99513a01daf33 Author: Hyunsik Choi <hyunsik@apache.org> Date: 2014-07-21T07:03:12Z TAJO-965 : Upgrade Bytes class and move some methods to others.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jinossy commented on the pull request:

        https://github.com/apache/tajo/pull/86#issuecomment-49626762

        Looks to clarify!! here is my +1

        Show
        githubbot ASF GitHub Bot added a comment - Github user jinossy commented on the pull request: https://github.com/apache/tajo/pull/86#issuecomment-49626762 Looks to clarify!! here is my +1
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user hyunsik commented on the pull request:

        https://github.com/apache/tajo/pull/86#issuecomment-49687799

        Thank you for quick review. I'll commit it shortly.

        Show
        githubbot ASF GitHub Bot added a comment - Github user hyunsik commented on the pull request: https://github.com/apache/tajo/pull/86#issuecomment-49687799 Thank you for quick review. I'll commit it shortly.
        Hide
        hyunsik Hyunsik Choi added a comment -

        committed.

        Show
        hyunsik Hyunsik Choi added a comment - committed.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

        https://github.com/apache/tajo/pull/86

        Show
        githubbot ASF GitHub Bot added a comment - Github user asfgit closed the pull request at: https://github.com/apache/tajo/pull/86
        Hide
        hudson Hudson added a comment -

        SUCCESS: Integrated in Tajo-master-build #308 (See https://builds.apache.org/job/Tajo-master-build/308/)
        TAJO-965: Upgrade Bytes class and move some methods to others. (hyunsik: rev 85b7031aa58971365a526c44c1b31c3385b7993d)

        • tajo-common/src/main/java/org/apache/tajo/util/Bytes.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/TextSerializerDeserializer.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/v2/RCFile.java
        • tajo-common/src/main/java/org/apache/tajo/util/BytesUtils.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/StorageUtil.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/RowFile.java
        • tajo-common/src/main/java/org/apache/tajo/util/NumberUtil.java
        • tajo-common/src/test/java/org/apache/tajo/util/TestBytes.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/index/bst/BSTIndex.java
        • tajo-common/src/test/java/org/apache/tajo/util/TestNumberUtil.java
        • tajo-storage/src/test/java/org/apache/tajo/storage/TestLazyTuple.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileAppender.java
        • tajo-common/src/test/java/org/apache/tajo/util/TestStringUtil.java
        • CHANGES
        • tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java
        • tajo-storage/src/main/java/org/apache/tajo/storage/v2/CSVFileScanner.java
        • tajo-core/src/test/java/org/apache/tajo/engine/eval/ExprTestBase.java
        Show
        hudson Hudson added a comment - SUCCESS: Integrated in Tajo-master-build #308 (See https://builds.apache.org/job/Tajo-master-build/308/ ) TAJO-965 : Upgrade Bytes class and move some methods to others. (hyunsik: rev 85b7031aa58971365a526c44c1b31c3385b7993d) tajo-common/src/main/java/org/apache/tajo/util/Bytes.java tajo-storage/src/main/java/org/apache/tajo/storage/TextSerializerDeserializer.java tajo-storage/src/main/java/org/apache/tajo/storage/v2/RCFile.java tajo-common/src/main/java/org/apache/tajo/util/BytesUtils.java tajo-storage/src/main/java/org/apache/tajo/storage/StorageUtil.java tajo-storage/src/main/java/org/apache/tajo/storage/RowFile.java tajo-common/src/main/java/org/apache/tajo/util/NumberUtil.java tajo-common/src/test/java/org/apache/tajo/util/TestBytes.java tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileScanner.java tajo-storage/src/main/java/org/apache/tajo/storage/index/bst/BSTIndex.java tajo-common/src/test/java/org/apache/tajo/util/TestNumberUtil.java tajo-storage/src/test/java/org/apache/tajo/storage/TestLazyTuple.java tajo-storage/src/main/java/org/apache/tajo/storage/rcfile/RCFile.java tajo-storage/src/main/java/org/apache/tajo/storage/sequencefile/SequenceFileAppender.java tajo-common/src/test/java/org/apache/tajo/util/TestStringUtil.java CHANGES tajo-storage/src/main/java/org/apache/tajo/storage/CSVFile.java tajo-storage/src/main/java/org/apache/tajo/storage/v2/CSVFileScanner.java tajo-core/src/test/java/org/apache/tajo/engine/eval/ExprTestBase.java

          People

          • Assignee:
            hyunsik Hyunsik Choi
            Reporter:
            hyunsik Hyunsik Choi
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development