>From Hussain Towaileb <[email protected]>:

Attention is currently required from: Hussain Towaileb.

Hello Anon. E. Moose #1000171, Jenkins,

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

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

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

The following approvals got outdated and were removed:
Contrib-2 by Anon. E. Moose #1000171, Integration-Tests+1 by Jenkins, 
Verified-1 by Jenkins


Change subject: [ASTERIXDB-3634][EXT]: correctly parse iceberg data types
......................................................................

[ASTERIXDB-3634][EXT]: correctly parse iceberg data types

Details:
- Parse float type as float instead of promoting to double.
- Parse Map type (with String key) to JSON object type.
- Ensure date, time and timestamp types are handled correctly.

Ext-ref: MB-70462
Change-Id: I968952f850e3e1c88d630566fb764f6b89ca2300
---
A asterixdb/asterix-app/data/json/iceberg_all_types.json
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/LocalCloudUtilAdobeMock.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/iceberg/IcebergTest.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.000.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.010.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.020.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.030.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.999.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-supported-catalog-types/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-supported-catalog-types/test.001.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/check-catalog-record/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/check-catalog-record/test.001.query.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/check-catalog-record/test.002.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/bad-catalog-type/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/bad-source-type/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/catalog-already-exists/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/catalog-already-exists/test.001.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/drop-does-not-exist/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/drop-has-dependent-collections/test.000.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/drop-has-dependent-collections/test.001.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/missing-source-type/test.000.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/iceberg/all-data-types/result.020.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/iceberg/all-data-types/result.030.adm
R 
asterixdb/asterix-app/src/test/resources/runtimets/results/iceberg/check-catalog-record/result.001.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_iceberg.xml
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/parser/IcebergParquetDataParser.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/aws/s3/S3Utils.java
28 files changed, 797 insertions(+), 276 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/26/21226/3
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21226?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: lumina
Gerrit-Change-Id: I968952f850e3e1c88d630566fb764f6b89ca2300
Gerrit-Change-Number: 21226
Gerrit-PatchSet: 3
Gerrit-Owner: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Hussain Towaileb <[email protected]>

Reply via email to