>From Hussain Towaileb <[email protected]>:

Attention is currently required from: Hussain Towaileb.

Hello Jenkins,

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

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

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


Change subject: [NO ISSUE][EXT]: fixes for timezone
......................................................................

[NO ISSUE][EXT]: fixes for timezone

Details:
- allow zone id values such as GMT+01:00
- when iceberg timezone is passed, only updated the UTC adjusted
  timestamp, the wallclock timestap should remain untouched.
- snapshots should not be impacted by provided timezone.
- use arithmetics for date claculation to avoid creation of objects
  unnecessarily.
- warn if decimal to double value hit infinity

Ext-ref: MB-72261
Change-Id: I2887ab0d9a79bf555a38596fdf941ff88d2e7aab
---
M 
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.050.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.060.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/all-data-types/test.999.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/invalid-timezone/test.000.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/invalid-timezone/test.010.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/iceberg/negative/invalid-timezone/test.999.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/iceberg/all-data-types/result.050.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/iceberg/all-data-types/result.060.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_iceberg.xml
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/input/record/reader/aws/iceberg/converter/IcebergConverterContext.java
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/ExternalDataUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/IcebergSnapshotUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/iceberg/IcebergUtils.java
15 files changed, 449 insertions(+), 51 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/14/21314/5
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21314?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: I2887ab0d9a79bf555a38596fdf941ff88d2e7aab
Gerrit-Change-Number: 21314
Gerrit-PatchSet: 5
Gerrit-Owner: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Attention: Hussain Towaileb <[email protected]>

Reply via email to