Repository: incubator-hawq Updated Branches: refs/heads/master 08a1c6c2a -> df6c3992d
HAWQ-1023. Fix incorrect usage of java.lang.String.replaceAll. Project: http://git-wip-us.apache.org/repos/asf/incubator-hawq/repo Commit: http://git-wip-us.apache.org/repos/asf/incubator-hawq/commit/df6c3992 Tree: http://git-wip-us.apache.org/repos/asf/incubator-hawq/tree/df6c3992 Diff: http://git-wip-us.apache.org/repos/asf/incubator-hawq/diff/df6c3992 Branch: refs/heads/master Commit: df6c3992d383dcd7433c12e8065c0d22806c6412 Parents: 08a1c6c Author: xunzhang <[email protected]> Authored: Sun Aug 28 12:49:30 2016 +0800 Committer: xunzhang <[email protected]> Committed: Sun Aug 28 12:49:30 2016 +0800 ---------------------------------------------------------------------- .../main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPath.java | 2 +- .../main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPoint.java | 2 +- .../main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPolygon.java | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/incubator-hawq/blob/df6c3992/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPath.java ---------------------------------------------------------------------- diff --git a/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPath.java b/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPath.java index ad553d3..4c2a4a0 100644 --- a/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPath.java +++ b/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPath.java @@ -48,7 +48,7 @@ public class HAWQPath public HAWQPath(String value) throws HAWQException { boolean open; - value.replaceAll(" ", ""); + value = value.replaceAll(" ", ""); if (value.startsWith("[") && value.endsWith("]")) { open = true; http://git-wip-us.apache.org/repos/asf/incubator-hawq/blob/df6c3992/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPoint.java ---------------------------------------------------------------------- diff --git a/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPoint.java b/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPoint.java index d248b59..354271c 100644 --- a/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPoint.java +++ b/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPoint.java @@ -42,7 +42,7 @@ public class HAWQPoint { if (value.startsWith("(") && value.endsWith(")")) { - value.replaceAll(" ", ""); + value = value.replaceAll(" ", ""); int posOfComma = value.indexOf(','); double x = Double.parseDouble(value.substring(1, posOfComma)); double y = Double.parseDouble(value.substring(posOfComma + 1, http://git-wip-us.apache.org/repos/asf/incubator-hawq/blob/df6c3992/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPolygon.java ---------------------------------------------------------------------- diff --git a/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPolygon.java b/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPolygon.java index 696de53..dc8ee94 100644 --- a/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPolygon.java +++ b/contrib/hawq-hadoop/hawq-mapreduce-common/src/main/java/com/pivotal/hawq/mapreduce/datatype/HAWQPolygon.java @@ -43,7 +43,7 @@ public class HAWQPolygon */ public HAWQPolygon(String value) throws HAWQException { - value.replaceAll(" ", ""); + value = value.replaceAll(" ", ""); String[] pointStrs = value.substring(1, value.length() - 1).split(","); if (pointStrs.length % 2 != 0) throw new HAWQException("Cannot convert " + value
