Merge pull request #78 from Esri/fromjson

JSON constructors getDisplayString (#77)
Esri · Apr 6, 2015 · 3b6e6a6 · 3b6e6a6
2 parents aee8226 + 92ad080
commit 3b6e6a6
Show file tree

Hide file tree

Showing 2 changed files with 22 additions and 8 deletions.
diff --git a/hive/src/main/java/com/esri/hadoop/hive/ST_GeomFromGeoJson.java b/hive/src/main/java/com/esri/hadoop/hive/ST_GeomFromGeoJson.java
@@ -53,11 +53,18 @@ public Object evaluate(DeferredObject[] arguments) throws HiveException {
 
 		return null;
 	}
-	
+
 	@Override
-	public String getDisplayString(String[] arg0) {
-		// TODO Auto-generated method stub
-		return "empty display string";
+	public String getDisplayString(String[] args) {
+		StringBuilder sb = new StringBuilder();
+		sb.append(this.getClass().getName());
+		String delim = "(";
+		for (String arg: args) {
+			sb.append(delim).append(arg);
+			delim = ", ";
+		}
+		sb.append(")");
+		return sb.toString();
 	}
 
 	@Override

diff --git a/hive/src/main/java/com/esri/hadoop/hive/ST_GeomFromJson.java b/hive/src/main/java/com/esri/hadoop/hive/ST_GeomFromJson.java
@@ -56,11 +56,18 @@ public Object evaluate(DeferredObject[] arguments) throws HiveException {
 
 		return null;
 	}
-	
+
 	@Override
-	public String getDisplayString(String[] arg0) {
-		// TODO Auto-generated method stub
-		return "empty display string";
+	public String getDisplayString(String[] args) {
+		StringBuilder sb = new StringBuilder();
+		sb.append(this.getClass().getName());
+		String delim = "(";
+		for (String arg: args) {
+			sb.append(delim).append(arg);
+			delim = ", ";
+		}
+		sb.append(")");
+		return sb.toString();
 	}
 
 	@Override