You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by "abdullah alamoudi (Code Review)" <do...@asterixdb.incubator.apache.org> on 2016/01/05 11:10:45 UTC
Change in asterixdb[master]: Add Support for Upsert Operation
abdullah alamoudi has posted comments on this change.
Change subject: Add Support for Upsert Operation
......................................................................
Patch Set 3:
(17 comments)
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
File asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java:
Line 90:
> WS
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-common/src/main/java/org/apache/asterix/common/feeds/api/IDatasourceAdapter.java
File asterix-common/src/main/java/org/apache/asterix/common/feeds/api/IDatasourceAdapter.java:
Line 53: */
> The comment seems wrong. Also, do we really need to extend the interface in
You are absolutely right. I have removed this.
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-external-data/src/main/java/org/apache/asterix/external/adapter/factory/PushBasedTwitterAdapterFactory.java
File asterix-external-data/src/main/java/org/apache/asterix/external/adapter/factory/PushBasedTwitterAdapterFactory.java:
Line 111: @Override
> Empty line above this?
This was removed :)
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-lang-aql/src/main/javacc/AQL.jj
File asterix-lang-aql/src/main/javacc/AQL.jj:
Line 884: upsert = InsertOrUpsert() "into" <DATASET> nameComponents = QualifiedName() query = Query()
> How about doing something like
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/AqlMetadataProvider.java
File asterix-metadata/src/main/java/org/apache/asterix/metadata/declared/AqlMetadataProvider.java:
Line 592: // ^^^^^^^^
> This looks scary ...
This is gone now with the external data cleanup.
Line 1273: // Done with this one.
> ??
Done
Line 1467: // The only remaining one.
> ??
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-metadata/src/main/java/org/apache/asterix/metadata/external/IAdapterFactory.java
File asterix-metadata/src/main/java/org/apache/asterix/metadata/external/IAdapterFactory.java:
Line 101: public void setFiles(List<ExternalFile> files) throws AlgebricksException;
> Do we need this on the interface? It seems that most implementations do not
Cleaned with the external data cleanup change.
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-metadata/src/main/java/org/apache/asterix/metadata/feeds/AbstractDatasourceAdapter.java
File asterix-metadata/src/main/java/org/apache/asterix/metadata/feeds/AbstractDatasourceAdapter.java:
Line 33: public abstract class AbstractDatasourceAdapter implements IDatasourceAdapter {
> Is this class used?
Cleaned up with external data change.
[Done]
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-metadata/src/main/java/org/apache/asterix/metadata/feeds/AbstractFeedDatasourceAdapter.java
File asterix-metadata/src/main/java/org/apache/asterix/metadata/feeds/AbstractFeedDatasourceAdapter.java:
Line 25: public abstract class AbstractFeedDatasourceAdapter implements IDatasourceAdapter {
> Is this class used?
removed with cleanup.
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/serde/ARecordSerializerDeserializer.java
File asterix-om/src/main/java/org/apache/asterix/dataflow/data/nontagged/serde/ARecordSerializerDeserializer.java:
Line 48: // This doesn't work well for deserialization. use ARecordPointable.
> ??
This class doesn't work for deserializing records :-) Thought a comment here is appropriate so no one would use this call.
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-runtime/src/main/java/org/apache/asterix/runtime/dataflow/AsterixLSMPrimaryUpsertOperatorNodePushable.java
File asterix-runtime/src/main/java/org/apache/asterix/runtime/dataflow/AsterixLSMPrimaryUpsertOperatorNodePushable.java:
Line 14: */
> This should have an ASF license.
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-runtime/src/main/java/org/apache/asterix/runtime/dataflow/AsterixLSMSecondaryUpsertOperatorNodePushable.java
File asterix-runtime/src/main/java/org/apache/asterix/runtime/dataflow/AsterixLSMSecondaryUpsertOperatorNodePushable.java:
Line 14: */
> ASF license
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/LockThenSearchOperationCallback.java
File asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/LockThenSearchOperationCallback.java:
Line 14: */
> ASF license
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/LockThenSearchOperationCallbackFactory.java
File asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/LockThenSearchOperationCallbackFactory.java:
Line 14: */
> ASF license
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/UpsertOperationCallback.java
File asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/UpsertOperationCallback.java:
Line 14: */
> ASF license
Done
https://asterix-gerrit.ics.uci.edu/#/c/477/3/asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/UpsertOperationCallbackFactory.java
File asterix-transactions/src/main/java/org/apache/asterix/transaction/management/opcallbacks/UpsertOperationCallbackFactory.java:
Line 14: */
> ASF license
Done
--
To view, visit https://asterix-gerrit.ics.uci.edu/477
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I8999000331795a5949d621d2dd003903e057a521
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: abdullah alamoudi <ba...@gmail.com>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-Reviewer: Taewoo Kim <wa...@gmail.com>
Gerrit-Reviewer: Till Westmann <ti...@apache.org>
Gerrit-Reviewer: Young-Seok Kim <ki...@gmail.com>
Gerrit-Reviewer: abdullah alamoudi <ba...@gmail.com>
Gerrit-HasComments: Yes