Skip to content

Commit 6199484

Browse files
authored
case test patches (#5531)
Signed-off-by: Simeon Widdis <sawiddis@amazon.com>
1 parent 30af8b2 commit 6199484

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

integ-test/src/test/java/org/opensearch/sql/calcite/remote/CalcitePPLCaseFunctionIT.java

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@
2525
import org.opensearch.sql.ppl.PPLIntegTestCase;
2626

2727
public class CalcitePPLCaseFunctionIT extends PPLIntegTestCase {
28+
2829
@Override
2930
public void init() throws Exception {
3031
super.init();
@@ -48,14 +49,15 @@ private void appendDataForBadResponse() throws IOException {
4849
new Request("PUT", "/" + TestsConstants.TEST_INDEX_WEBLOGS + "/_doc/8?refresh=true");
4950
request2.setJsonEntity(
5051
"{\"host\": \"0.0.0.2\", \"method\": \"GET\", \"url\":"
51-
+ " \"/shuttle/missions/sts-73/mission-sts-73.html\", \"response\": \"500\", \"bytes\":"
52-
+ " \"4085\"}");
52+
+ " \"/shuttle/missions/sts-73/mission-sts-73.html\", \"response\": \"500\","
53+
+ " \"bytes\": \"4085\"}");
5354
client().performRequest(request2);
5455
Request request3 =
5556
new Request("PUT", "/" + TestsConstants.TEST_INDEX_WEBLOGS + "/_doc/9?refresh=true");
5657
request3.setJsonEntity(
57-
"{\"host\": \"::3\", \"method\": \"GET\", \"url\": \"/shuttle/countdown/countdown.html\","
58-
+ " \"response\": \"403\", \"bytes\": \"3985\"}");
58+
"{\"host\": \"::3\", \"method\": \"GET\", \"url\":"
59+
+ " \"/shuttle/countdown/countdown.html\", \"response\": \"403\", \"bytes\":"
60+
+ " \"3985\"}");
5961
client().performRequest(request3);
6062
Request request4 =
6163
new Request("PUT", "/" + TestsConstants.TEST_INDEX_WEBLOGS + "/_doc/10?refresh=true");
@@ -308,7 +310,7 @@ public void testCaseCanBePushedDownAsRangeQuery() throws IOException {
308310
TEST_INDEX_BANK));
309311
verifySchema(actual4, schema("avg(balance)", "double"), schema("age_range", "string"));
310312
// There's such a discrepancy because null cannot be the key for a range query
311-
if (isPushdownDisabled()) {
313+
if (isPushdownDisabled() || isAnalyticsParquetIndicesEnabled()) {
312314
verifyDataRows(
313315
actual4,
314316
rows(32838.0, "u30"),
@@ -463,7 +465,7 @@ public void testCaseAggWithNullValues() throws IOException {
463465
TEST_INDEX_STATE_COUNTRY_WITH_NULL));
464466
verifySchema(actual, schema("avg(age)", "double"), schema("age_category", "string"));
465467
// There is such discrepancy because range aggregations will ignore null values
466-
if (isPushdownDisabled()) {
468+
if (isPushdownDisabled() || isAnalyticsParquetIndicesEnabled()) {
467469
verifyDataRows(
468470
actual,
469471
rows(10, "teenager"),

0 commit comments

Comments
 (0)