Github user chenghao-intel commented on a diff in the pull request:
https://github.com/apache/spark/pull/4562#discussion_r24797498
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/parquet/parquetSuites.scala ---
@@ -121,13 +121,50 @@ class ParquetDataSourceOnMetastoreSuite extends
ParquetMetastoreSuiteBase {
override def beforeAll(): Unit = {
super.beforeAll()
+
+ sql(s"""
+ create table test_parquet
+ (
+ intField INT,
+ stringField STRING
+ )
+ ROW FORMAT SERDE
'org.apache.hadoop.hive.ql.io.parquet.serde.ParquetHiveSerDe'
+ STORED AS
+ INPUTFORMAT
'org.apache.hadoop.hive.ql.io.parquet.MapredParquetInputFormat'
+ OUTPUTFORMAT
'org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat'
+ """)
+
+ val rdd = sparkContext.parallelize((1 to 10).map(i => s"""{"a":$i,
"b":"str${i}"}"""))
+ jsonRDD(rdd).registerTempTable("jt")
+ sql("""
+ create table test ROW FORMAT
+ | SERDE
'org.apache.hadoop.hive.ql.io.parquet.serde.ParquetHiveSerDe'
+ | STORED AS INPUTFORMAT
'org.apache.hadoop.hive.ql.io.parquet.MapredParquetInputFormat'
+ | OUTPUTFORMAT
'org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat'
+ | AS select * from jt""".stripMargin)
+
--- End diff --
Oh, i thought `STORED AS PARQUERT AS ..` is just the syntactic sugar.
Unfortunately, all of the test suite are implemented in the sub project `sql`,
but the `HiveShim` is in the subproject `hive` with `hive` package accessing
visibility.
Let's put this test in another PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
