>From Ian Maxon <ima...@apache.org>:

Attention is currently required from: Ian Maxon.

Hello Anon. E. Moose #1000171, Jenkins,

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

    https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20280?usp=email

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

The following approvals got outdated and were removed:
Contrib-2 by Anon. E. Moose #1000171, Integration-Tests+1 by Jenkins, 
Verified-1 by Jenkins


Change subject: [ASTERIXDB-3640][TEST]Use s3mock in testcontainers
......................................................................

[ASTERIXDB-3640][TEST]Use s3mock in testcontainers

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

Details:
- Refactor all tests that use Adobe S3 Mock to use
  the containerized version instead of using it
  as a Java dependency

- Make the cloud storage configuration templated
  to deal with the fact testcontainers listens
  on a random ephemeral port each time

- Add compensating drop statements to some test
  suites that were previously leaking entities
  into the Default namespace

Ext-ref: MB-68268
Change-Id: Iad1a6f9331fd844f3e3c04b8f56e811e4117f83a
---
M asterixdb/asterix-app/pom.xml
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/api/common/LocalCloudUtilAdobeMock.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/atomic_statements/AtomicMetadataTransactionWithoutWALTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/atomic_statements/AtomicStatementsCancellationTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/CloudPythonTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/CloudStorageAzTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/CloudStorageGCSTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/CloudStorageSparseTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/CloudStorageTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/CloudStorageUnstableTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/cloud_storage/GCSCloudStorageUnstableTest.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestConstants.java
M 
asterixdb/asterix-app/src/test/java/org/apache/asterix/test/common/TestExecutor.java
R asterixdb/asterix-app/src/test/resources/cc-cloud-storage-sparse.conf.ftl
C asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf.ftl
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/api/post-non-query/post-non-query.099.post.http
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/array_fun/array_slice/array_slice_bracket_notation/array_slice_bracket_notation.099.ddl.sqlpp
M 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/cloud_storage/storage-size/datasource-function/external-collection/test.010.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/join/hash_join_exchange/hash_join_exchange.099.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/join/hash_join_exchange_1/hash_join_exchange_1.099.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/join/join-with-empty-dataset/join-with-empty-dataset.099.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/misc/limit_after_offset/limit_after_offset.099.ddl.sqlpp
R 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/misc/limit_after_offset/limit_after_offset.1.ddl.sqlpp
C 
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/misc/limit_after_offset/limit_after_offset.2.query.sqlpp
A 
asterixdb/asterix-app/src/test/resources/runtimets/results/api/post-non-query/post-non-query.099.adm
M asterixdb/asterix-app/src/test/resources/runtimets/testsuite_cloud_storage.xml
M asterixdb/pom.xml
27 files changed, 193 insertions(+), 109 deletions(-)


  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/80/20280/15
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20280?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: phoenix
Gerrit-Change-Id: Iad1a6f9331fd844f3e3c04b8f56e811e4117f83a
Gerrit-Change-Number: 20280
Gerrit-PatchSet: 15
Gerrit-Owner: Ian Maxon <ima...@apache.org>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
Gerrit-CC: Michael Blow <mb...@apache.org>
Gerrit-Attention: Ian Maxon <ima...@apache.org>

Reply via email to