You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by sh...@apache.org on 2019/11/08 00:45:51 UTC
[samza] 01/01: Revert "Fixed typo in SqlFieldSchema::getValueSchema
method name (#1205)"
This is an automated email from the ASF dual-hosted git repository.
shanthoosh pushed a commit to branch revert-1205-fix_typo_in_SqlFieldSchema
in repository https://gitbox.apache.org/repos/asf/samza.git
commit def0bb940c59072ebe2c403617eaa2c6e9c02302
Author: shanthoosh <sv...@linkedin.com>
AuthorDate: Thu Nov 7 16:45:44 2019 -0800
Revert "Fixed typo in SqlFieldSchema::getValueSchema method name (#1205)"
This reverts commit 5defb06702d8a2d216eb07df2ba2a079731233af.
---
samza-api/src/main/java/org/apache/samza/sql/schema/SqlFieldSchema.java | 2 +-
.../main/java/org/apache/samza/sql/client/impl/CliCommandHandler.java | 2 +-
.../src/main/java/org/apache/samza/sql/planner/RelSchemaConverter.java | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/samza-api/src/main/java/org/apache/samza/sql/schema/SqlFieldSchema.java b/samza-api/src/main/java/org/apache/samza/sql/schema/SqlFieldSchema.java
index 7c4fd32..d3cec05 100644
--- a/samza-api/src/main/java/org/apache/samza/sql/schema/SqlFieldSchema.java
+++ b/samza-api/src/main/java/org/apache/samza/sql/schema/SqlFieldSchema.java
@@ -97,7 +97,7 @@ public class SqlFieldSchema {
/**
* Get the schema of the value if the field type is {@link SamzaSqlFieldType#MAP}
*/
- public SqlFieldSchema getValueSchema() {
+ public SqlFieldSchema getValueScehma() {
return valueType;
}
diff --git a/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/CliCommandHandler.java b/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/CliCommandHandler.java
index c18c637..bb594e1 100644
--- a/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/CliCommandHandler.java
+++ b/samza-sql-shell/src/main/java/org/apache/samza/sql/client/impl/CliCommandHandler.java
@@ -671,7 +671,7 @@ public class CliCommandHandler implements CommandHandler {
return String.format("ARRAY(%s)", getFieldDisplayValue(fieldSchema.getElementSchema()));
case MAP:
return String.format("MAP(%s, %s)", SamzaSqlFieldType.STRING.toString(),
- getFieldDisplayValue(fieldSchema.getValueSchema()));
+ getFieldDisplayValue(fieldSchema.getValueScehma()));
case ROW:
String rowDisplayValue = fieldSchema.getRowSchema()
.getFields()
diff --git a/samza-sql/src/main/java/org/apache/samza/sql/planner/RelSchemaConverter.java b/samza-sql/src/main/java/org/apache/samza/sql/planner/RelSchemaConverter.java
index c2653d9..7df5916 100644
--- a/samza-sql/src/main/java/org/apache/samza/sql/planner/RelSchemaConverter.java
+++ b/samza-sql/src/main/java/org/apache/samza/sql/planner/RelSchemaConverter.java
@@ -98,7 +98,7 @@ public class RelSchemaConverter extends SqlTypeFactoryImpl {
// TODO Calcite execution engine doesn't support record type yet.
return createTypeWithNullability(createSqlType(SqlTypeName.ANY), true);
case MAP:
- RelDataType valueType = getRelDataType(fieldSchema.getValueSchema());
+ RelDataType valueType = getRelDataType(fieldSchema.getValueScehma());
return new MapSqlType(createSqlType(SqlTypeName.VARCHAR), valueType, true);
default:
String msg = String.format("Field Type %s is not supported", fieldSchema.getFieldType());