You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2023/10/23 22:28:42 UTC
Change in asterixdb[master]: [WIP][ASTERIXDB-3284][IDX] Prevent insertion of invalid array index e...
From Glenn Galvizo <gg...@uci.edu>:
Attention is currently required from: Glenn Galvizo.
Hello Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17875
to look at the new patch set (#3).
Change subject: [WIP][ASTERIXDB-3284][IDX] Prevent insertion of invalid array index entries
......................................................................
[WIP][ASTERIXDB-3284][IDX] Prevent insertion of invalid array index entries
- user model changes: no
- storage format changes: no
- interface changes: no
details:
preventing the insertion of missing / null values into an array index.
also added a cast-lax immediately before the index maintenance op to
cast incorrectly typed values into missing.
Change-Id: I8e88ef93e5e2210f78112dd35bd75a40dfa6300e
---
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryUpsertWithNestedPlanOperatorDescriptor.java
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorNodePushable.java
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryUpsertWithNestedPlanOperatorNodePushable.java
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryUpsertOperatorDescriptor.java
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryUpsertOperatorNodePushable.java
M hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/operators/logical/IndexInsertDeleteUpsertOperator.java
M asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/IntroduceSecondaryIndexInsertDeleteRule.java
M asterixdb/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/MetadataProvider.java
M asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/operators/LSMSecondaryInsertDeleteWithNestedPlanOperatorDescriptor.java
9 files changed, 113 insertions(+), 66 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/75/17875/3
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17875
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I8e88ef93e5e2210f78112dd35bd75a40dfa6300e
Gerrit-Change-Number: 17875
Gerrit-PatchSet: 3
Gerrit-Owner: Glenn Galvizo <gg...@uci.edu>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Attention: Glenn Galvizo <gg...@uci.edu>
Gerrit-MessageType: newpatchset