Change in asterixdb[master]: Refactor QueryTranslator

[Murtadha Hubail (Code Review)]

Murtadha Hubail has abandoned this change.

Change subject: Refactor QueryTranslator
..


Abandoned

Maybe one day!

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: abandon
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 5: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/797/ : 
SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 5:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/797/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-Reviewer: abdullah alamoudi 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 5:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2852/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 5
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Murtadha Hubail (Code Review)]

Hello Jenkins,

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

https://asterix-gerrit.ics.uci.edu/1215

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

Change subject: Refactor QueryTranslator
..

Refactor QueryTranslator

This change moves the implementation of each statement to the statement itself
instead of having all implementations in the QueryTranslator. Some statements
were not refactored in this change due to projects dependency issues or they
need special handling in the QueryTranslator itself.

Change-Id: I732b063be8b733af504cbefc9844721953c30621
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutorFactory.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ConnectorAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryWebInterfaceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/VersionAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/java/AsterixJavaClient.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/FeedWorkCollection.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultPrinter.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultReader.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/GlobalRecoveryManager.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/bootstrap/TestNodeController.java
R 
asterixdb/asterix-app/src/test/java/org/apache/asterix/aql/translator/RunStatementTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
R asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Job.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/AsterixRuntimeComponentsProvider.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/AsterixAppContextInfo.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobSpecificationUtils.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IExternalDataSourceFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/feed/management/FeedEventsListener.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java
M 
asterixdb/asterix-lang-aql/src/main/java/org/apache/asterix/lang/aql/statement/SubscribeFeedStatement.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/CompiledStatements.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/ExternalIndexingOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IStatementExecutor.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IndexOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/SecondaryBTreeOperationsHelper.java
R 

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 4:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2851/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 4
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Murtadha Hubail (Code Review)]

Hello Jenkins,

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

https://asterix-gerrit.ics.uci.edu/1215

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

Change subject: Refactor QueryTranslator
..

Refactor QueryTranslator

This change moves the implementation of each statement to the statement itself
instead of having all implementations in the QueryTranslator. Some statements
were not refactored in this change due to projects dependency issues or they
need special handling in the QueryTranslator itself.

Change-Id: I732b063be8b733af504cbefc9844721953c30621
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutorFactory.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ConnectorAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryWebInterfaceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/VersionAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/java/AsterixJavaClient.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/FeedWorkCollection.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultPrinter.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultReader.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/GlobalRecoveryManager.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/bootstrap/TestNodeController.java
R 
asterixdb/asterix-app/src/test/java/org/apache/asterix/aql/translator/RunStatementTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
R asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Job.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/AsterixRuntimeComponentsProvider.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/AsterixAppContextInfo.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobSpecificationUtils.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IExternalDataSourceFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/feed/management/FeedEventsListener.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java
M 
asterixdb/asterix-lang-aql/src/main/java/org/apache/asterix/lang/aql/statement/SubscribeFeedStatement.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/CompiledStatements.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/ExternalIndexingOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IStatementExecutor.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IndexOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/SecondaryBTreeOperationsHelper.java
R 

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 3: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/781/ : 
SUCCESS

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 3:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/781/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 3:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2831/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 3
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Murtadha Hubail (Code Review)]

Hello Jenkins,

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

https://asterix-gerrit.ics.uci.edu/1215

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

Change subject: Refactor QueryTranslator
..

Refactor QueryTranslator

This change moves the implementation of each statement to the statement itself
instead of having all implementations in the QueryTranslator. Some statements
were not refactored in this change due to projects dependency issues or they
need special handling in the QueryTranslator itself.

Change-Id: I732b063be8b733af504cbefc9844721953c30621
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutorFactory.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ConnectorAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryWebInterfaceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/VersionAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/java/AsterixJavaClient.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/FeedWorkCollection.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultPrinter.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultReader.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/GlobalRecoveryManager.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/bootstrap/TestNodeController.java
R 
asterixdb/asterix-app/src/test/java/org/apache/asterix/aql/translator/RunStatementTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
R asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Job.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/AsterixRuntimeComponentsProvider.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/AsterixAppContextInfo.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobSpecificationUtils.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IExternalDataSourceFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/feed/management/FeedEventsListener.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java
M 
asterixdb/asterix-lang-aql/src/main/java/org/apache/asterix/lang/aql/statement/SubscribeFeedStatement.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/CompiledStatements.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/ExternalIndexingOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IStatementExecutor.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IndexOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/SecondaryBTreeOperationsHelper.java
R 

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 2: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/760/ : 
UNSTABLE

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 2:

Integration Tests Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/760/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 2:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2807/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 2
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Murtadha Hubail (Code Review)]

Hello Jenkins,

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

https://asterix-gerrit.ics.uci.edu/1215

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

Change subject: Refactor QueryTranslator
..

Refactor QueryTranslator

This change moves the implementation of each statement to the statement itself
instead of having all implementations in the QueryTranslator. Some statements
were not refactored in this change due to projects dependency issues or they
need special handling in the QueryTranslator itself.

Change-Id: I732b063be8b733af504cbefc9844721953c30621
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutorFactory.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ConnectorAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryWebInterfaceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/VersionAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/java/AsterixJavaClient.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/FeedWorkCollection.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultPrinter.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultReader.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/GlobalRecoveryManager.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/bootstrap/TestNodeController.java
R 
asterixdb/asterix-app/src/test/java/org/apache/asterix/aql/translator/RunStatementTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
R asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Job.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/AsterixRuntimeComponentsProvider.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/AsterixAppContextInfo.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobSpecificationUtils.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IExternalDataSourceFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/feed/management/FeedEventsListener.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java
M 
asterixdb/asterix-lang-aql/src/main/java/org/apache/asterix/lang/aql/statement/SubscribeFeedStatement.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/CompiledStatements.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/ExternalIndexingOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IStatementExecutor.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IndexOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/SecondaryBTreeOperationsHelper.java
R 

Change in asterixdb[master]: Refactor QueryTranslator

[Jenkins (Code Review)]

Jenkins has posted comments on this change.

Change subject: Refactor QueryTranslator
..


Patch Set 1:

Build Started 
https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-notopic/2798/

-- 
To view, visit https://asterix-gerrit.ics.uci.edu/1215
To unsubscribe, visit https://asterix-gerrit.ics.uci.edu/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I732b063be8b733af504cbefc9844721953c30621
Gerrit-PatchSet: 1
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Owner: Murtadha Hubail 
Gerrit-Reviewer: Jenkins 
Gerrit-HasComments: No

Change in asterixdb[master]: Refactor QueryTranslator

[Murtadha Hubail (Code Review)]

Murtadha Hubail has uploaded a new change for review.

  https://asterix-gerrit.ics.uci.edu/1215

Change subject: Refactor QueryTranslator
..

Refactor QueryTranslator

Change-Id: I732b063be8b733af504cbefc9844721953c30621
---
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/base/ILangExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/extension/IExtensionStatement.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/algebra/operators/physical/InvertedIndexPOperator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AbstractLangTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/AqlPlusExpressionToPlanTranslator.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutorFactory.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/LangExpressionToPlanTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/APIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/ConnectorAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryResultAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryServiceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/QueryWebInterfaceServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/RESTAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/servlet/VersionAPIServlet.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/java/AsterixJavaClient.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/external/FeedWorkCollection.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultPrinter.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultReader.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResultUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplicationEntryPoint.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/GlobalRecoveryManager.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/http/servlet/VersionAPIServletTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/app/bootstrap/TestNodeController.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/aql/translator/QueryTranslatorTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/ExecutionTestUtil.java
R asterixdb/asterix-common/src/main/java/org/apache/asterix/common/api/Job.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/AsterixPropertiesAccessor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/AsterixReplicationProperties.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/transactions/AsterixRuntimeComponentsProvider.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/AsterixAppContextInfo.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobSpecificationUtils.java
R 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/JobUtils.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/api/IExternalDataSourceFactory.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/feed/management/FeedEventsListener.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/library/ExternalFunction.java
M 
asterixdb/asterix-external-data/src/main/java/org/apache/asterix/external/util/HDFSUtils.java
M 
asterixdb/asterix-lang-aql/src/main/java/org/apache/asterix/lang/aql/statement/SubscribeFeedStatement.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/CompiledStatements.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/ExternalIndexingOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IStatementExecutor.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/IndexOperations.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/SecondaryBTreeOperationsHelper.java
R 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/base/SecondaryIndexOperationsHelper.java
R