Michael Blow has uploaded a new change for review.
https://asterix-gerrit.ics.uci.edu/1084
Change subject: Fix Numeric Overflow For Units > GB
......................................................................
Fix Numeric Overflow For Units > GB
Change-Id: Id5a3896ebfb38fc565b3524beed940dfbbf81d4f
---
M
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/StorageUtil.java
1 file changed, 3 insertions(+), 3 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/84/1084/1
diff --git
a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/StorageUtil.java
b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/StorageUtil.java
index 7025f15..c33d33e 100644
---
a/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/StorageUtil.java
+++
b/hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/StorageUtil.java
@@ -20,7 +20,7 @@
public class StorageUtil {
- private static final int BASE = 1024, KB = BASE, MB = KB * BASE, GB = MB *
BASE, TB = GB * BASE, PB = TB * BASE;
+ private static final long BASE = 1024, KB = BASE, MB = KB * BASE, GB = MB
* BASE, TB = GB * BASE, PB = TB * BASE;
public enum StorageUnit {
BYTE("B"),
@@ -87,7 +87,7 @@
* Helper method to parse a byte unit string to its double value and unit
* (e.g., 10,345.8MB becomes Pair<10345.8, StorageUnit.MB>.)
*
- * @throws HyracksException
+ * @throws IllegalStateException
*/
public static ByteValueStringInfo parseByteUnitString(String s) {
String sSpaceRemoved = s.replaceAll(" ", "");
@@ -146,7 +146,7 @@
/**
* Return byte value for the given string (e.g., 0.1KB, 100kb, 1mb, 3MB,
8.5GB ...)
*
- * @throws HyracksException
+ * @throws IllegalStateException
*/
public static long getByteValue(String s) {
try {
--
To view, visit https://asterix-gerrit.ics.uci.edu/1084
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id5a3896ebfb38fc565b3524beed940dfbbf81d4f
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Michael Blow <[email protected]>
