juliuszsompolski commented on a change in pull request #29539:
URL: https://github.com/apache/spark/pull/29539#discussion_r478286103
##########
File path:
sql/hive-thriftserver/src/test/scala/org/apache/spark/sql/hive/thriftserver/HiveThriftServer2Suites.scala
##########
@@ -686,6 +686,23 @@ class HiveThriftBinaryServerSuite extends
HiveThriftJdbcTest {
}
}
+ test("Query Intervals in VIEWs through thrift server") {
+ val viewName1 = "view_interval_1"
+ val viewName2 = "view_interval_2"
+ val ddl1 = s"CREATE GLOBAL TEMP VIEW $viewName1 AS SELECT INTERVAL 1 DAY
AS i"
+ val ddl2 = s"CREATE TEMP VIEW $viewName2 as select * from
global_temp.$viewName1"
+ withJdbcStatement(viewName1, viewName2) { statement =>
+ statement.executeQuery(ddl1)
+ statement.executeQuery(ddl2)
+ val rs = statement.executeQuery(s"SELECT v1.i as a, v2.i as b FROM
global_temp.$viewName1" +
Review comment:
In ResultSetMetadata would the client-side user not learn that they
should follow string rules by using getColumnClassName, which should return
String, while it would be still valuable to them to know that that column
represents an interval?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
