Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-05-01 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 21: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/435/ : SUCCESS


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 21
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Wed, 01 May 2024 19:35:48 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-05-01 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 21: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/uSWZVjKzs9SyJ5sv9 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 21
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Wed, 01 May 2024 18:25:39 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-05-01 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#21) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-30 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 20: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/425/ : SUCCESS


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 20
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Wed, 01 May 2024 01:57:01 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-30 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 20: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/bdR4ezCkpJYd8kJd7 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 20
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 30 Apr 2024 20:40:46 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-30 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#20) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-30 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 19: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/418/ : SUCCESS


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 19
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 30 Apr 2024 19:07:28 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-30 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 19: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/WDkUutVj5BXUHDQJ7 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 19
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 30 Apr 2024 18:11:46 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-30 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#19) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 18: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/411/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 18
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 30 Apr 2024 03:45:16 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 18: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/e82ifDM8u71GXTqA9 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 18
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 30 Apr 2024 01:52:51 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#18) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 17: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/zHTvZykh3Vy9jbhg6 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 17
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 23:44:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#17) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 16: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/405/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 16
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 22:38:58 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 16: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/w3gryt2EVLCBTDto8 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 16
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 20:50:28 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#16) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 15: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/UuorYsUsmKpdxTQJ7 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 15
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 19:57:11 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#15) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 14: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/mQHBHZqvKdY5kgXJA : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 14
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 18:36:15 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#14) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 13: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/396/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 13
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 17:15:24 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 13: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/Wbgp8a2mH2eYdRNQ8 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 13
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 16:36:30 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-29 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#13) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.08.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.06.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.03.regexadm
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.001.regexjson
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/warnings/warnings-limit/warnings-limit.04.regexadm
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/request-param-validation-400-BAD/request-param-validation-400-BAD.01.regexjson
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.05.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cache-residency/cache-residency.002.regexjson
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
M 
asterixdb/asterix-app/src/test/resources/runtimets/results_cbo/warnings/warnings-limit/warnings-limit.04.regexadm
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-28 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 12: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/393/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 12
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 04:58:39 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-28 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 12: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/MAio2Ssjep6WCNVs9 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 12
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 29 Apr 2024 03:50:13 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-28 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#12) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/fields/MetricsPrinter.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/IStatementExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/translator/QueryTranslator.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.001.regexjson
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.004.json
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.006.json
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.003.post.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/result/ResponseMetrics.java
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.004.delete.http
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.006.delete.http
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.007.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceServlet.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.002.post.http
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RebalanceApiServlet.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryResultApiServlet.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.005.post.http
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.007.json
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/dataflow/ICcApplicationContext.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.003.regexjson
M 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-28 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 11: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/392/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 11
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Sun, 28 Apr 2024 20:07:31 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-28 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 11: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/SaQBDqzeKRGHyChB8 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 11
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Sun, 28 Apr 2024 19:27:28 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-04-28 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#11) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.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/api/http/server/QueryServiceServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RebalanceApiServlet.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.2.adm
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.1.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/dataflow/ICcApplicationContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceRequestParameters.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/PlanCacheApiServlet.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/SqlCompatRewriteVisitor.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/PredicateCardinalityAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
A 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppQueryCachePrintVisitor.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.3.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.1.delete.http
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.2.query.sqlpp
M 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-02-01 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 10: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/77/ : SUCCESS


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 10
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 01 Feb 2024 18:05:45 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-02-01 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 10: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/dbjSZ3zUVcTmHLNc7 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 10
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 01 Feb 2024 17:04:26 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 9: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/29/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 9
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 22 Jan 2024 05:24:07 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 9: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/sQeK8hFxREpeu63r8 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 9
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Mon, 22 Jan 2024 04:55:20 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#9) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/warnings/warnings-limit/warnings-limit.08.post.http
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/OptimizationConfUtil.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.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/api/http/server/QueryServiceServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RebalanceApiServlet.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.2.adm
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/warnings/warnings-limit/warnings-limit.03.post.http
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.1.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/warnings/warnings-limit/warnings-limit.05.post.http
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/dataflow/ICcApplicationContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceRequestParameters.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/PlanCacheApiServlet.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/warnings/warnings-limit/warnings-limit.07.post.http
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/SqlCompatRewriteVisitor.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/warnings/warnings-limit/warnings-limit.04.post.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/PredicateCardinalityAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
A 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 8:

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/27/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 8
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Sun, 21 Jan 2024 21:47:25 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 8: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/26/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 8
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Sun, 21 Jan 2024 21:10:01 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 8: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/miDENqgccTPV4qJd6 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 8
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Sun, 21 Jan 2024 20:42:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-21 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#8) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: yes

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M asterixdb/asterix-app/src/test/resources/runtimets/api.xml
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/clear/ClearCachePredicate.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/api/http/server/QueryServiceServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/hyracks/bootstrap/CCApplication.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RebalanceApiServlet.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.2.adm
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/plan_cache_clear/plan_cache_clear.1.json
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/utils/Servlets.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/dataflow/ICcApplicationContext.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceRequestParameters.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/PlanCacheApiServlet.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/SqlCompatRewriteVisitor.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
asterixdb/asterix-test-framework/src/main/java/org/apache/asterix/testframework/context/TestCaseContext.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/PredicateCardinalityAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
A 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppQueryCachePrintVisitor.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.3.delete.http
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.1.delete.http
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
A 
asterixdb/asterix-app/src/test/resources/runtimets/queries/api/plan_cache_clear/plan_cache_clear.2.query.sqlpp
M 

Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Integration-Tests-1

Integration Tests Failed

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/2/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Fri, 19 Jan 2024 07:52:57 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-trigger/1/ : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Fri, 19 Jan 2024 07:09:05 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/2125/ : null


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Fri, 19 Jan 2024 06:41:15 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: -Integration-Tests

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Fri, 19 Jan 2024 06:35:16 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Fri, 19 Jan 2024 06:33:47 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/2118/ : null


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:33:06 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: -Integration-Tests

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:27:10 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-integration-tests/2117/ : null


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:19:06 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:13:09 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:11:44 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:06:24 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 19:00:40 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 18:59:53 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 18:58:06 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-gerrit-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 18:57:25 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 18:56:31 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has removed a vote from this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )


Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Removed Integration-Tests+1 by Jenkins 
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-MessageType: deleteVote


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Ian Maxon :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Ian Maxon has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

(1 comment)

Patchset:

PS7:
TRIGGER asterix-gerrit-integration-tests



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-CC: Ian Maxon 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 18:54:46 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/13425/ 
: null


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 18:05:20 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-18 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: -Integration-Tests

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 17:59:11 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-17 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Integration-Tests+1

Integration Tests Successful

https://asterix-jenkins.ics.uci.edu/job/asterix-gerrit-integration-tests/13424/ 
: null


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 07:27:44 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-17 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 07:21:12 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-17 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 06:02:17 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-17 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Thu, 18 Jan 2024 00:11:01 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-16 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/mqtCj9q9Ue2HDg4v9 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 16 Jan 2024 22:24:10 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-16 Thread AsterixDB Code Review
>From Jenkins :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Jenkins has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 7:

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


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Tue, 16 Jan 2024 22:22:16 +
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-16 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#7) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: no

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.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/api/http/server/QueryServiceServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RebalanceApiServlet.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceRequestParameters.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/upsert/upsert-case-returning/upsert-case-returning.3.query.sqlpp
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/SqlCompatRewriteVisitor.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/PredicateCardinalityAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/visitor/QueryPrintVisitor.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/JoinProductivityAnnotation.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dml/insert-return-records/insert-return-records.3.query.sqlpp
29 files changed, 583 insertions(+), 14 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/20/17620/7
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 7
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-MessageType: newpatchset


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-12 Thread AsterixDB Code Review
Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Anon. E. Moose #1000171 has posted comments on this change. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..


Patch Set 6: Contrib-2

Analytics Compatibility Compilation Failed
https://cbjenkins.page.link/n8dDfahXJZXWNWRf7 : UNSTABLE


--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 6
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-Comment-Date: Fri, 12 Jan 2024 21:47:12 +
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


Change in asterixdb[master]: [ASTERIXDB-3183][COMP] Add query plan cache

2024-01-12 Thread AsterixDB Code Review
>From Glenn Galvizo :

Attention is currently required from: Sushrut Borkar, Glenn Galvizo.
Glenn Galvizo has uploaded a new patch set (#6) to the change originally 
created by Sushrut Borkar. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620 )

Change subject: [ASTERIXDB-3183][COMP] Add query plan cache
..

[ASTERIXDB-3183][COMP] Add query plan cache

- user model changes: yes
- storage format changes: no
- interface changes: no

Details:

The query plan cache reduces compile times by storing job specs for
old queries. This allows skipping the rewriting and compilation
stages of repeated queries.

Change-Id: I87813c761f5024742fb8658360166613d01f127d
---
M 
hyracks-fullstack/hyracks/hyracks-control/hyracks-control-common/src/main/java/org/apache/hyracks/control/common/controllers/CCConfig.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/api/http/server/QueryServiceServlet.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheKey.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/HashJoinExpressionAnnotation.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCacheValue.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/annotation/ExcludeFromSelectStarAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/LangExecutionUtil.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/RebalanceApiServlet.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/BroadcastExpressionAnnotation.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql-sugar/avg_mixed/avg_mixed.5.query.sqlpp
A 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/runtime/SqlppExecutionWithCacheTest.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/QueryServiceRequestParameters.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/translator/SessionConfig.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/upsert/upsert-case-returning/upsert-case-returning.3.query.sqlpp
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/rewrites/visitor/SqlCompatRewriteVisitor.java
A 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cache/QueryCache.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/app/cc/CcApplicationContext.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/PredicateCardinalityAnnotation.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
M 
asterixdb/asterix-app/src/main/java/org/apache/asterix/api/common/APIFramework.java
M 
asterixdb/asterix-lang-sqlpp/src/main/java/org/apache/asterix/lang/sqlpp/visitor/SqlppAstPrintVisitor.java
M 
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/annotations/ExternalSubpathAnnotation.java
M 
asterixdb/asterix-lang-common/src/main/java/org/apache/asterix/lang/common/visitor/QueryPrintVisitor.java
M 
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/algebra/expressions/JoinProductivityAnnotation.java
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/dml/insert-return-records/insert-return-records.3.query.sqlpp
29 files changed, 585 insertions(+), 13 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/20/17620/6
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17620
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: I87813c761f5024742fb8658360166613d01f127d
Gerrit-Change-Number: 17620
Gerrit-PatchSet: 6
Gerrit-Owner: Sushrut Borkar 
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Glenn Galvizo 
Gerrit-Reviewer: Jenkins 
Gerrit-Attention: Sushrut Borkar 
Gerrit-Attention: Glenn Galvizo 
Gerrit-MessageType: newpatchset