Yingyi Bu has posted comments on this change. Change subject: Support Change Feeds and Ingestion of Records with MetaData ......................................................................
Patch Set 14: (4 comments) https://asterix-gerrit.ics.uci.edu/#/c/621/14/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/MetaFunctionToMetaVariableRule.java File asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/MetaFunctionToMetaVariableRule.java: Line 265: exprRef.setValue(new VariableReferenceExpression(keyVars.get(i))); This seems not working when you have composite keys... There are two options: 1. We can give key() an integer parameter i, to indicate it's the i-th key variable. 2. I think it's totally fine that for now, we do not allow composite keys, and then you can change the list keyVars here to a LogicalVariable keyVar and throw exceptions when you see composite keys. I prefer option 2 but either way sounds good to me. https://asterix-gerrit.ics.uci.edu/#/c/621/14/asterix-algebra/src/main/java/org/apache/asterix/translator/util/PlanTranslationUtil.java File asterix-algebra/src/main/java/org/apache/asterix/translator/util/PlanTranslationUtil.java: Line 45: @SuppressWarnings("unchecked") You no longer need this class? createFieldAcess, etc? https://asterix-gerrit.ics.uci.edu/#/c/621/14/asterix-external-data/src/main/java/org/apache/asterix/external/api/IDataFlowController.java File asterix-external-data/src/main/java/org/apache/asterix/external/api/IDataFlowController.java: Line 26: public void start(IFrameWriter writer) throws HyracksDataException; remove IFrameWriter? (You don't need to do that if you feel that's too much for this change.) https://asterix-gerrit.ics.uci.edu/#/c/621/14/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/FeedDataSource.java File asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/FeedDataSource.java: Line 45: private final LogicalVariable metaVar; remove metaVar and pkVars? -- To view, visit https://asterix-gerrit.ics.uci.edu/621 To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings Gerrit-MessageType: comment Gerrit-Change-Id: If136a03d424970132dfb09f0dda56e160d4c0078 Gerrit-PatchSet: 14 Gerrit-Project: asterixdb Gerrit-Branch: master Gerrit-Owner: abdullah alamoudi <[email protected]> Gerrit-Reviewer: Ildar Absalyamov <[email protected]> Gerrit-Reviewer: Jenkins <[email protected]> Gerrit-Reviewer: Steven Jacobs <[email protected]> Gerrit-Reviewer: Till Westmann <[email protected]> Gerrit-Reviewer: Yingyi Bu <[email protected]> Gerrit-Reviewer: abdullah alamoudi <[email protected]> Gerrit-HasComments: Yes
