>From Hussain Towaileb <[email protected]>:

Attention is currently required from: Hussain Towaileb, Michael Blow.

Hello Jenkins, Michael Blow, 

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

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

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

The following approvals got outdated and were removed:
Code-Review+1 by Hussain Towaileb, Integration-Tests+1 by Jenkins, Verified+1 
by Jenkins


Change subject: [NO ISSUE][EXT]: fix parsing nulls and handling time micros for 
avro
......................................................................

[NO ISSUE][EXT]: fix parsing nulls and handling time micros for avro

Details:
- Properly report Avro NULL value as null rather than missing and
  omitting the whole field.
- Stop applying the timezone offset to time-of-day and local
  timestamp values, which have no associated timezone; only
  UTC timestamp-millis/micros values get the offset.

Ext-ref: MB-72320, MB-72680
Change-Id: I872be48898e01e7eefe4f1e2c85e72f1ca0c9c2e
---
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/external_dataset/avro/AvroLogicalTypesExampleGenerator.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/external-dataset/common/avro/avro-logical-types/avro-logical-types.01.ddl.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/external-dataset/common/avro/avro-logical-types/avro-logical-types.04.query.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/array-access/array-access.02.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/array-access/array-access.03.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/avro-logical-types/avro-logical-types.02.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/avro-logical-types/avro-logical-types.03.adm
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/avro-logical-types/avro-logical-types.04.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/select-all-fields/select-all-fields.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/external-dataset/common/avro/type-mismatch/type-mismatch.02.adm
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/parser/AvroDataParser.java
M 
hyracks-fullstack/hyracks/hyracks-util/src/main/java/org/apache/hyracks/util/annotations/AiProvenance.java
12 files changed, 121 insertions(+), 36 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/89/21389/2
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21389?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: master
Gerrit-Change-Id: I872be48898e01e7eefe4f1e2c85e72f1ca0c9c2e
Gerrit-Change-Number: 21389
Gerrit-PatchSet: 2
Gerrit-Owner: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Hussain Towaileb <[email protected]>
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Michael Blow <[email protected]>
Gerrit-Attention: Michael Blow <[email protected]>
Gerrit-Attention: Hussain Towaileb <[email protected]>

Reply via email to