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

TB and PB representations in StorageUnit are overflow

    Details

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

      Description

      See the StorageUnit class. TB and PB are overflow against integer presentation.

      public class StorageUnit {
      
      	public static final int B = 8;
      	public static final int KB = 1024;
      	public static final int MB = KB * 1024;
      	public static final int GB = MB * 1024;
      	public static final int TB = GB * 1024;
      	public static final int PB = TB * 1024;
      }
      

        Activity

        Hide
        githubbot ASF GitHub Bot added a comment -

        GitHub user ykrips opened a pull request:

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

        TAJO-1299: TB and PB representations in StorageUnit are overflow

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

        $ git pull https://github.com/ykrips/tajo TAJO-1299

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

        https://github.com/apache/tajo/pull/348.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 #348


        commit 960bdb799dfc7f4d3a512aa95b7308bd3882b1a6
        Author: Jihun Kang <jihun@apache.org>
        Date: 2015-01-12T09:20:59Z

        TAJO-1299: TB and PB representations in StorageUnit are overflow


        Show
        githubbot ASF GitHub Bot added a comment - GitHub user ykrips opened a pull request: https://github.com/apache/tajo/pull/348 TAJO-1299 : TB and PB representations in StorageUnit are overflow You can merge this pull request into a Git repository by running: $ git pull https://github.com/ykrips/tajo TAJO-1299 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/tajo/pull/348.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 #348 commit 960bdb799dfc7f4d3a512aa95b7308bd3882b1a6 Author: Jihun Kang <jihun@apache.org> Date: 2015-01-12T09:20:59Z TAJO-1299 : TB and PB representations in StorageUnit are overflow
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user jihoonson commented on the pull request:

        https://github.com/apache/tajo/pull/348#issuecomment-69693135

        +1
        ship it!

        Show
        githubbot ASF GitHub Bot added a comment - Github user jihoonson commented on the pull request: https://github.com/apache/tajo/pull/348#issuecomment-69693135 +1 ship it!
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user ykrips commented on the pull request:

        https://github.com/apache/tajo/pull/348#issuecomment-69695658

        Hello @jihoonson ,
        Thank you for reviewing this patch.

        Show
        githubbot ASF GitHub Bot added a comment - Github user ykrips commented on the pull request: https://github.com/apache/tajo/pull/348#issuecomment-69695658 Hello @jihoonson , Thank you for reviewing this patch.
        Hide
        githubbot ASF GitHub Bot added a comment -

        Github user asfgit closed the pull request at:

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

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

        ABORTED: Integrated in Tajo-master-CODEGEN-build #193 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/193/)
        TAJO-1299: TB and PB representations in StorageUnit are overflow (jihun: rev 9553edc9ba0d6a0d4ff6e75b6be5d06aef91eddb)

        • CHANGES
        • tajo-common/src/main/java/org/apache/tajo/unit/StorageUnit.java
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-CODEGEN-build #193 (See https://builds.apache.org/job/Tajo-master-CODEGEN-build/193/ ) TAJO-1299 : TB and PB representations in StorageUnit are overflow (jihun: rev 9553edc9ba0d6a0d4ff6e75b6be5d06aef91eddb) CHANGES tajo-common/src/main/java/org/apache/tajo/unit/StorageUnit.java
        Hide
        hudson Hudson added a comment -

        ABORTED: Integrated in Tajo-master-build #554 (See https://builds.apache.org/job/Tajo-master-build/554/)
        TAJO-1299: TB and PB representations in StorageUnit are overflow (jihun: rev 9553edc9ba0d6a0d4ff6e75b6be5d06aef91eddb)

        • tajo-common/src/main/java/org/apache/tajo/unit/StorageUnit.java
        • CHANGES
        Show
        hudson Hudson added a comment - ABORTED: Integrated in Tajo-master-build #554 (See https://builds.apache.org/job/Tajo-master-build/554/ ) TAJO-1299 : TB and PB representations in StorageUnit are overflow (jihun: rev 9553edc9ba0d6a0d4ff6e75b6be5d06aef91eddb) tajo-common/src/main/java/org/apache/tajo/unit/StorageUnit.java CHANGES

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development