[L] Change in asterixdb[phoenix]: [ASTERIXDB-3392] Default to STRING for null-typed fields during sche...

[AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu>]

From Preetham Poluparthi <pr...@apache.org>:

Attention is currently required from: Ali Alsuliman, Preetham Poluparthi.

Hello Ali Alsuliman, Anon. E. Moose #1000171, Jenkins, 

I'd like you to reexamine a change. Please visit

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20395?usp=email

to look at the new patch set (#6).

The following approvals got outdated and were removed:
Integration-Tests+1 by Jenkins


Change subject: [ASTERIXDB-3392]  Default to STRING for null-typed fields during schema inference in COPY TO parquet
......................................................................

[ASTERIXDB-3392]  Default to STRING for null-typed fields during schema inference in COPY TO parquet

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
When a compute partition consists entirely of null values, schema inference in copy to parquet defaults to STRING type in parquet.

Ext-ref: MB-68610
Change-Id: I60069bf3d44c60effb69e5555bbf7e869327cc5b
---
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/negative/parquet-error-checks/parquet-error-checks.23.ddl.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null1/parquet-null1.01.ddl.sqlpp
R asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null1/parquet-null1.02.update.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null1/parquet-null1.03.ddl.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null1/parquet-null1.04.query.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null2/parquet-null2.01.ddl.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null2/parquet-null2.02.update.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null2/parquet-null2.03.ddl.sqlpp
C asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-null2/parquet-null2.04.query.sqlpp
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-simple/parquet-simple.01.ddl.sqlpp
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/copy-to/parquet-simple/parquet-simple.02.update.sqlpp
A asterixdb/asterix-app/src/test/resources/runtimets/results/copy-to/parquet-null1/parquet-null1.04.adm
A asterixdb/asterix-app/src/test/resources/runtimets/results/copy-to/parquet-null2/parquet-null2.04.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_external_dataset_s3.xml
M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/writer/printer/parquet/ParquetSchemaTree.java
M asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/writer/printer/parquet/SchemaCheckerLazyVisitor.java
16 files changed, 105 insertions(+), 150 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/95/20395/6
-- 
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20395?usp=email
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: phoenix
Gerrit-Change-Id: I60069bf3d44c60effb69e5555bbf7e869327cc5b
Gerrit-Change-Number: 20395
Gerrit-PatchSet: 6
Gerrit-Owner: Preetham Poluparthi <pr...@apache.org>
Gerrit-Reviewer: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Attention: Ali Alsuliman <al...@gmail.com>
Gerrit-Attention: Preetham Poluparthi <pr...@apache.org>