Skip to content

Commit d5aea8c

Browse files
committed
Minor cleanup for PR review
1 parent be26238 commit d5aea8c

File tree

2 files changed

+2
-3
lines changed

2 files changed

+2
-3
lines changed

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/EsqlSession.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -327,7 +327,6 @@ public void analyzedPlan(
327327
var listener = SubscribableListener.<EnrichResolution>newForked(
328328
l -> enrichPolicyResolver.resolvePolicies(targetClusters, unresolvedPolicies, l)
329329
).<PreAnalysisResult>andThen((l, enrichResolution) -> resolveFieldNames(parsed, enrichResolution, l));
330-
331330
// first resolve the lookup indices, then the main indices
332331
for (TableInfo lookupIndex : preAnalysis.lookupIndices) {
333332
listener = listener.andThen((l, preAnalysisResult) -> { preAnalyzeLookupIndex(lookupIndex, preAnalysisResult, l); });

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/EsqlSessionCCSUtils.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -199,8 +199,8 @@ static void updateExecutionInfoWithClustersWithNoMatchingIndices(EsqlExecutionIn
199199

200200
/**
201201
* Rules enforced at planning time around non-matching indices
202-
* 1. fail query if no matching indices on any cluster (VerificationException) - that is handled elsewhere (TODO: document where)
203-
* 2. fail query if a cluster has no matching indices *and* a concrete index was specified
202+
* 1. fail query if no matching indices on any cluster (VerificationException) - that is handled elsewhere
203+
* 2. fail query if a cluster has no matching indices *and* a concrete index was specified - handled here
204204
*/
205205
String fatalErrorMessage = null;
206206
/*

0 commit comments

Comments
 (0)