-
- Downloads
[SPARK-16590][SQL] Improve LogicalPlanToSQLSuite to check generated SQL directly
## What changes were proposed in this pull request? This PR improves `LogicalPlanToSQLSuite` to check the generated SQL directly by **structure**. So far, `LogicalPlanToSQLSuite` relies on `checkHiveQl` to ensure the **successful SQL generation** and **answer equality**. However, it does not guarantee the generated SQL is the same or will not be changed unnoticeably. ## How was this patch tested? Pass the Jenkins. This is only a testsuite change. Author: Dongjoon Hyun <dongjoon@apache.org> Closes #14235 from dongjoon-hyun/SPARK-16590.
Showing
- sql/hive/src/test/resources/sqlgen/agg1.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/agg1.sql
- sql/hive/src/test/resources/sqlgen/agg2.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/agg2.sql
- sql/hive/src/test/resources/sqlgen/agg3.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/agg3.sql
- sql/hive/src/test/resources/sqlgen/aggregate_functions_and_window.sql 4 additions, 0 deletions.../test/resources/sqlgen/aggregate_functions_and_window.sql
- sql/hive/src/test/resources/sqlgen/case.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/case.sql
- sql/hive/src/test/resources/sqlgen/case_with_else.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/case_with_else.sql
- sql/hive/src/test/resources/sqlgen/case_with_key.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/case_with_key.sql
- sql/hive/src/test/resources/sqlgen/case_with_key_and_else.sql 4 additions, 0 deletions...hive/src/test/resources/sqlgen/case_with_key_and_else.sql
- sql/hive/src/test/resources/sqlgen/cluster_by.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/cluster_by.sql
- sql/hive/src/test/resources/sqlgen/data_source_json_parquet_t0.sql 4 additions, 0 deletions...src/test/resources/sqlgen/data_source_json_parquet_t0.sql
- sql/hive/src/test/resources/sqlgen/data_source_orc_parquet_t0.sql 4 additions, 0 deletions.../src/test/resources/sqlgen/data_source_orc_parquet_t0.sql
- sql/hive/src/test/resources/sqlgen/data_source_parquet_parquet_t0.sql 4 additions, 0 deletions.../test/resources/sqlgen/data_source_parquet_parquet_t0.sql
- sql/hive/src/test/resources/sqlgen/distinct_aggregation.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/distinct_aggregation.sql
- sql/hive/src/test/resources/sqlgen/distribute_by.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/distribute_by.sql
- sql/hive/src/test/resources/sqlgen/distribute_by_with_sort_by.sql 4 additions, 0 deletions.../src/test/resources/sqlgen/distribute_by_with_sort_by.sql
- sql/hive/src/test/resources/sqlgen/except.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/except.sql
- sql/hive/src/test/resources/sqlgen/filter_after_subquery.sql 4 additions, 0 deletionssql/hive/src/test/resources/sqlgen/filter_after_subquery.sql
- sql/hive/src/test/resources/sqlgen/generate_with_other_1.sql 8 additions, 0 deletionssql/hive/src/test/resources/sqlgen/generate_with_other_1.sql
- sql/hive/src/test/resources/sqlgen/generate_with_other_2.sql 10 additions, 0 deletionssql/hive/src/test/resources/sqlgen/generate_with_other_2.sql
- sql/hive/src/test/resources/sqlgen/generator_in_lateral_view_1.sql 4 additions, 0 deletions...src/test/resources/sqlgen/generator_in_lateral_view_1.sql
Loading
Please register or sign in to comment