Skip to content

Commit 077facf

Browse files
committed
Fix spotless issues and yaml merge issue
1 parent 46c4a16 commit 077facf

File tree

6 files changed

+47
-16
lines changed

6 files changed

+47
-16
lines changed

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/LongDoubleBucketedSort.java

Lines changed: 8 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/LongFloatBucketedSort.java

Lines changed: 8 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/LongIntBucketedSort.java

Lines changed: 8 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/LongLongBucketedSort.java

Lines changed: 7 additions & 2 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/sort/X-BucketedSort.java.st

Lines changed: 16 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -9,10 +9,10 @@ package org.elasticsearch.compute.data.sort;
99

1010
import org.elasticsearch.common.util.BigArrays;
1111
import org.elasticsearch.common.util.BitArray;
12-
import org.elasticsearch.common.util.$Type$Array;
1312
$if(hasExtra)$
1413
import org.elasticsearch.common.util.$ExtraType$Array;
1514
$endif$
15+
import org.elasticsearch.common.util.$Type$Array;
1616
import org.elasticsearch.common.util.PageCacheRecycler;
1717
import org.elasticsearch.compute.data.Block;
1818
import org.elasticsearch.compute.data.BlockFactory;
@@ -67,11 +67,13 @@ public class $Name$$ExtraName$BucketedSort implements Releasable {
6767
* </li>
6868
* </ul>
6969
*/
70-
private $Array$ values;
7170
$if(hasExtra)$
71+
private $Array$ values;
7272
private $ExtraArray$ extraValues;
73-
$endif$
7473

74+
$else$
75+
private $Array$ values;
76+
$endif$
7577

7678
public $Name$$ExtraName$BucketedSort(BigArrays bigArrays, SortOrder order, int bucketSize) {
7779
this.bigArrays = bigArrays;
@@ -446,7 +448,17 @@ $endif$
446448
}
447449
int rightChild = leftChild + 1;
448450
long rightIndex = rootIndex + rightChild;
449-
if (rightChild < heapSize && betterThan(values.get(worstIndex), values.get(rightIndex)$if(hasExtra)$, extraValues.get(worstIndex), extraValues.get(rightIndex)$endif$)) {
451+
$if(hasExtra)$
452+
if (rightChild < heapSize
453+
&& betterThan(
454+
values.get(worstIndex),
455+
values.get(rightIndex),
456+
extraValues.get(worstIndex),
457+
extraValues.get(rightIndex)
458+
)) {
459+
$else$
460+
if (rightChild < heapSize && betterThan(values.get(worstIndex), values.get(rightIndex))) {
461+
$endif$
450462
worst = rightChild;
451463
worstIndex = rightIndex;
452464
}

x-pack/plugin/src/yamlRestTest/resources/rest-api-spec/test/esql/60_usage.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -133,7 +133,6 @@ setup:
133133
# Testing for the entire function set isn't feasible, so we just check that we return the correct count as an approximation.
134134
- length: {esql.functions: 180} # check the "sister" test below for a likely update to the same esql.functions length check
135135

136-
>>>>>>> 75a450da0c4 (Add SPARKLINE aggregation fuction)
137136
---
138137
"Basic ESQL usage output (telemetry) non-snapshot version":
139138
- requires:

0 commit comments

Comments
 (0)