opendev/gerrit
Code Issues Proposed changes

ElasticV{6,7}QueryChangesTest: Add comment about issue 10120

Browse Source 
Add a comment to clarify that the index close operation is done
to prevent issue 10120.

Change-Id: Id8d5b4e922674404351ac9a5a7a281283485fdb1
This commit is contained in:
David Pursehouse
2020-04-02 22:35:07 +09:00
parent 6e8615737f
commit 5495d9cc5b
2 changed files with 4 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
javatests/com/google/gerrit/elasticsearch/ElasticV6QueryChangesTest.java
View File

@@ -57,6 +57,8 @@ public class ElasticV6QueryChangesTest extends AbstractQueryChangesTest {
@After @After
public void closeIndex() { public void closeIndex() {
// Close the index after each test to prevent exceeding Elasticsearch's
// shard limit (see Issue 10120).
client.execute( client.execute(
new HttpPost( new HttpPost(
String.format( String.format(

2
javatests/com/google/gerrit/elasticsearch/ElasticV7QueryChangesTest.java
View File

@@ -57,6 +57,8 @@ public class ElasticV7QueryChangesTest extends AbstractQueryChangesTest {
@After @After
public void closeIndex() { public void closeIndex() {
// Close the index after each test to prevent exceeding Elasticsearch's
// shard limit (see Issue 10120).
client.execute( client.execute(
new HttpPost( new HttpPost(
String.format( String.format(