server/src/main/java/org/elasticsearch/snapshots/SnapshotsService.java (16 lines): - line 316: // TODO: It is worth revisiting the design choice of creating a placeholder entry in snapshots-in-progress here once we have a cache - line 327: // TODO: create snapshot UUID in CloneSnapshotRequest and make this operation idempotent to cleanly deal with transport layer - line 476: // TODO: we could skip this step for snapshots with state SUCCESS - line 496: EsExecutors.DIRECT_EXECUTOR_SERVICE, // TODO contemplate threading here, do we need to fork, see #101445? - line 560: // TODO: If we could eventually drop the snapshot clone init phase we don't need this any longer - line 1020: // TODO: the code in this state update duplicates large chunks of the logic in #SHARD_STATE_EXECUTOR. - line 1273: // TODO: Restart snapshot on another node? - line 1373: EsExecutors.DIRECT_EXECUTOR_SERVICE, // TODO contemplate threading here, do we need to fork, see #101445? - line 1435: // BlobStoreRepository#finalizeSnapshot is kind of awkward and artificial; TODO consolidate all this stuff into one place and simplify - line 1726: * TODO: optimize this to execute in a single CS update together with finalizing the latest snapshot - line 2449: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 2540: // TODO we should complete the deletion and resolve the listeners of SnapshotDeletionsInProgress with no snapshot sooner, - line 2765: // TODO: be more efficient here, we could collect newly ready shard clones as we compute them and then directly start them - line 2814: // TODO: dry up redundant computation and code between clone and non-clone case, in particular reuse - line 2826: // TODO: the below logic is very similar to that in #startCloning and both could be dried up against each other - line 3094: // TODO multi-project: this will behave incorrectly when there are data streams with equal names in different projects that are server/src/main/java/org/elasticsearch/repositories/blobstore/BlobStoreRepository.java (14 lines): - line 569: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 939: * TODO: Add shard level cleanups - line 940: * TODO: Add unreferenced index metadata cleanup - line 1129: // TODO should we even write the new RepositoryData unless all shard paths have been successfully updated? See #100569. - line 1199: // TODO should we pass newRepositoryData to cleanupStaleBlobs()? - line 1265: // TODO Skip this unnecessary work? Maybe track the shard count in RepositoryData? - line 1285: // TODO: Should we fail the delete here? See https://github.com/elastic/elasticsearch/issues/100569. - line 1309: // Computed at the start of doRun(), after forking, rather than in the constructor. TODO make these local variables perhaps? - line 1421: // TODO: Should we fail the delete here? See https://github.com/elastic/elasticsearch/issues/100569. - line 1576: // TODO: Include the current generation here once we remove keeping index-(N-1) around from #writeIndexGen - line 1825: } // else this task was largely a no-op - TODO no need to fork in that case - line 1845: // TODO fail fast if any metadata write fails - line 1846: // TODO clean up successful metadata writes on failure (needs care, we must not clobber another node concurrently - line 3277: // TODO apparently we don't use the MetadataSnapshot#.recoveryDiff(...) here but we should x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene40/blocktree/SegmentTermsEnumFrame.java (10 lines): - line 183: // TODO: if suffixes were stored in random-access - line 243: // TODO: we could skip this if !hasTerms; but - line 340: // TODO: make this array'd so we can do bin search? - line 425: // TODO: better API would be "jump straight to term=N"??? - line 428: // TODO: we could make "tiers" of metadata, ie, - line 434: // TODO: if docFreq were bulk decoded we could - line 569: // TODO: binary search when all terms have the same length, which is common for ID fields, - line 637: // TODO: not consistent that in the - line 714: // TODO this - line 757: // TODO: not consistent that in the x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/EsqlSession.java (9 lines): - line 207: // TODO: this could be snuck into the underlying listener - line 239: // TODO: merge into one method - line 333: (String) e.policyName().fold(FoldContext.small() /* TODO remove me*/), - line 356: // TODO in follow-PR (for skip_unavailable handling of missing concrete indexes) add some tests for - line 398: // TODO: Verify that the resolved index actually has indexMode: "lookup" - line 428: // TODO we plan to support joins in the future when possible, but for now we'll just fail early if we see one - line 447: // TODO: Maybe if no indices are returned, retry without index mode and provide a clearer error message. - line 745: // TODO: have an ESFilter and push down to EsQueryExec / EsSource - line 747: // TODO: filter integration testing x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/generative/EsqlQueryGenerator.java (9 lines): - line 37: // TODO re-enable ROW. - line 87: // TODO more complex conditions - line 109: // TODO LIKE, RLIKE, functions etc. - line 140: // TODO add WITH - line 145: // TODO make it smarter and extend it to other types - line 400: // TODO pass newly created fields to next expressions - line 468: // TODO more numerics - line 508: // TODO improve!!! - line 555: // TODO not only simple values, but also foldable expressions x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authc/Authentication.java (8 lines): - line 253: // TODO how can this not be true - line 443: // TODO: should we validate enable status and length of role names on instantiation time of anonymousUser? - line 507: // TODO: this is not entirely accurate if anonymous user can create a token - line 1243: // TODO is a newer version than the node's a valid value? - line 1254: // TODO assert SystemUser.is(fallbackUser); - line 1273: // TODO make the service account user a separate class/interface - line 1283: // TODO make the type system ensure that this is not a run-as user - line 1588: // TODO: Rename to AuthenticationMethod x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/FST.java (8 lines): - line 60: // TODO: we can free up a bit if we can nuke this: - line 542: // TODO: really we should encode this as an arc, arriving - line 685: // TODO: for better perf (but more RAM used) we - line 748: // TODO: try to avoid wasteful cases: disable doFixedLengthArcs in that case - line 1301: // TODO: can't assert this because we call from readFirstArc - line 1361: // TODO: would be nice to make this lazy -- maybe - line 1397: // TODO: could we somehow [partially] tableize arc lookups - line 1488: // TODO: we should fix this code to not have to create x-pack/plugin/esql/arrow/src/main/java/org/elasticsearch/xpack/esql/arrow/BlockConverter.java (8 lines): - line 104: // TODO could we "just" get the memory of the array and dump it? - line 142: // TODO could we "just" get the memory of the array and dump it? - line 183: // TODO could we "just" get the memory of the array and dump it? - line 268: // TODO could we "just" get the memory of the array and dump it? - line 273: // FIXME: add a ByteRefsVector.getLength(position): there are some cases - line 291: // TODO could we "just" get the memory of the array and dump it? - line 313: // TODO we can probably get the length from the vector without all this sum - line 526: // TODO could we get the block's firstValueIndexes and dump it? server/src/main/java/org/elasticsearch/snapshots/RestoreService.java (8 lines): - line 368: // TODO: https://github.com/elastic/elasticsearch/issues/119545 - This does not yet support selectors - line 574: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 1608: // TODO: Should all existing templates be deleted first? - line 1622: // TODO: Check request.skipOperatorOnly for Autoscaling policies (NonRestorableCustom) - line 1624: // TODO: Should we restore them at the end? - line 1684: // TODO: Enable restore for open indices - line 1817: // TODO: automatically add runtime field definitions for source-only snapshots - line 1858: // TODO: _routing? Perhaps we don't need to obey any routing here as stuff is read-only anyway and get API will be disabled server/src/main/java/org/elasticsearch/index/engine/InternalEngine.java (8 lines): - line 160: // TODO: move the following two to the stateless plugin - line 905: * TODO: ideally we should keep around translog entries long enough to cover this case - line 1449: // TODO: can we enable this check for all origins except primary on the leader? - line 2109: // TODO: maybe we should just put a scheduled job in threadPool? - line 2138: // TODO: it might be more relevant to use the RAM usage of the largest DWPT as opposed to the overall RAM usage? Can we get this - line 2477: // TODO: Migrate to using async apic - line 2535: // TODO: Split acquireLastIndexCommit into two apis one with blocking flushes one without - line 2770: // TODO: Introduce an index setting for setMaxFullFlushMergeWaitMillis server/src/main/java/org/elasticsearch/index/codec/vectors/es816/BinaryQuantizer.java (8 lines): - line 271: // FIXME: make a copy of vector so we don't overwrite it here? - line 281: // FIXME: quantize these values so we are passing back 1 byte values for all three of these - line 290: // FIXME: quantize these values so we are passing back 1 byte values for all three of these - line 305: // FIXME: speed up with panama? and/or use existing scalar quantization utils in Lucene? - line 348: // FIXME: make a copy of vector so we don't overwrite it here? - line 352: // FIXME: should other similarity functions behave like MIP on query like COSINE - line 374: // FIXME: quantize the corrections as well so we store less - line 377: // FIXME: quantize the corrections as well so we store less x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/analysis/Analyzer.java (7 lines): - line 237: // TODO: fix this (and tests), or drop check (seems SQL-inherited, where's also defective) - line 316: final String policyName = (String) plan.policyName().fold(FoldContext.small() /* TODO remove me */); - line 608: // TODO: remove this when Stats interface is removed - line 1540: value.fold(FoldContext.small() /* TODO remove me */), - line 1544: return new UnresolvedAttribute(value.source(), String.valueOf(value.fold(FoldContext.small() /* TODO remove me */)), message); - line 1549: TemporalAmount result = maybeParseTemporalAmount(from.fold(FoldContext.small() /* TODO remove me */).toString().strip()); - line 1567: EsqlDataTypeConverter.convert(from.fold(FoldContext.small() /* TODO remove me */), target), server/src/main/java/org/elasticsearch/cluster/metadata/DataStream.java (7 lines): - line 257: // TODO: clear out the failure_store field, which is redundant https://github.com/elastic/elasticsearch/issues/127071 - line 426: // TODO: make index_mode, start and end time fields in IndexMetadata class. - line 1298: in.readString(); // TODO: clear out the timestamp field, which is a constant https://github.com/elastic/elasticsearch/issues/101991 - line 1313: out.writeString(TIMESTAMP_FIELD_NAME); // TODO: clear this out in the future https://github.com/elastic/elasticsearch/issues/101991 - line 1331: // TODO: clear out the failure_store field, which is redundant https://github.com/elastic/elasticsearch/issues/127071 - line 1366: // TODO: clear out the failure_store field, which is redundant https://github.com/elastic/elasticsearch/issues/127071 - line 1436: // TODO: clear out the failure_store field, which is redundant https://github.com/elastic/elasticsearch/issues/127071 server/src/main/java/org/elasticsearch/index/engine/Engine.java (7 lines): - line 135: public static final String SEARCH_SOURCE = "search"; // TODO: Make source of search enum? - line 859: // TODO: A better exception goes here - line 1316: // TODO: add more fine grained mem stats values to per segment info here - line 1392: // TODO: Remove or rename for increased clarity - line 2141: // TODO remove this blocking, or at least do it elsewhere, see https://github.com/elastic/elasticsearch/issues/89821 - line 2171: // TODO: We are not waiting for full durability here atm because we are on the cluster state update thread - line 2285: // TODO make all the production usages fully async server/src/main/java/org/elasticsearch/common/Rounding.java (6 lines): - line 699: // TODO this is used in date range's collect so we should optimize it too - line 749: // TODO this is actually used date range's collect so we should optimize it - line 1124: // TODO this is used in date range's collect so we should optimize it too - line 1154: // TODO this is used in date range's collect so we should optimize it too - line 1254: return offsetInstant.toEpochMilli(); // TODO or throw something? - line 1406: // TODO we can likely translate here server/src/internalClusterTest/java/org/elasticsearch/versioning/SimpleVersioningIT.java (6 lines): - line 179: // TODO: This behavior breaks rest api returning http status 201 - line 278: // TODO: ADD SEQ NO! - line 506: // TODO: sometimes use _bulk API - line 507: // TODO: test non-aborting exceptions (Rob suggested field where positions overflow) - line 509: // TODO: not great we don't test deletes GC here: - line 595: // TODO: sometimes use bulk: x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/predicate/operator/comparison/EsqlBinaryComparison.java (6 lines): - line 176: // TODO: We are definitely not doing the right thing with this zoneId - line 190: // TODO this uses a constructor on the operation *and* a name which is confusing. It only needs one. Everything else uses a name. - line 195: // TODO: Remove zoneId entirely - line 381: Object value = valueOf(FoldContext.small() /* TODO remove me */, right()); - line 395: // TODO: This type coersion layer is copied directly from the QL counterpart code. It's probably not necessary or desireable - line 481: Object value = valueOf(FoldContext.small() /* TODO remove me */, right()); x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/repository/CcrRepository.java (6 lines): - line 262: // TODO stop doing this as a blocking activity - line 263: // TODO on timeout, cancel the remote request, don't just carry on - line 264: // TODO handle exceptions better, don't just unwrap/rewrap them with actionGet - line 430: // TODO: Add timeouts to network calls / the restore process. - line 479: // TODO: There should be some local timeout. And if the remote cluster returns an unknown session - line 669: threadPool.generic(), // TODO should be the remote-client response executor to match the non-timeout case x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/FieldExtractorTestCase.java (6 lines): - line 286: // TODO we should support loading geo_point from doc values if source isn't enabled - line 299: // TODO if source isn't enabled how can we load *something*? It's just triangles, right? - line 303: // TODO pick supported random shapes - line 338: // TODO this is broken in main too - line 1352: // TODO these should all always return null because the parent is nested - line 1418: /* TODO add support to this test for disabling _source server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/LongKeyedBucketOrds.java (6 lines): - line 325: // TODO it'd be faster to count the number of buckets in a list of these ords rather than one at a time - line 342: // TODO this is fairly expensive to compute. Can we avoid needing it? - line 357: // TODO it'd be faster to iterate many ords at once rather than one at a time - line 452: // TODO it'd be faster to count the number of buckets in a list of these ords rather than one at a time - line 473: // TODO this is fairly expensive to compute. Can we avoid needing it? - line 488: // TODO it'd be faster to iterate many ords at once rather than one at a time build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/ElasticsearchTestBasePlugin.java (6 lines): - line 98: // TODO remove once jvm.options are added to test system properties - line 115: // TODO: only open these for mockito when it is modularized - line 178: // TODO: remove setting logging level via system property - line 181: // TODO: remove this once ctx isn't added to update script params in 7.0 - line 184: // TODO: remove this property in 8.0 - line 187: // TODO: remove this once cname is prepended to transport.publish_address by default in 8.0 server/src/main/java/org/elasticsearch/cluster/metadata/IndexMetadata.java (6 lines): - line 377: * to retain compatibility with old indexes. TODO: remove in 9.0. - line 514: * TODO: Remove in 9.0 - line 526: * TODO: Remove in 9.0 - line 1593: // TODO: find a nicer way to do BwC here and just work with Diff here and in networking - line 1862: // TODO: adjust serialization format to using an optional writable - line 2689: // TODO: do this in 6.0: x-pack/plugin/core/src/main/java/org/elasticsearch/license/ClusterStateLicenseService.java (5 lines): - line 209: // TODO: ack messages should be generated on the master, since another node's cluster state may be behind... - line 269: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 313: startBasicTaskQueue.submitTask(task.getDescription(), task, null); // TODO should pass in request.masterNodeTimeout() here - line 344: null // TODO should pass in request.masterNodeTimeout() here - line 358: startBasicTaskQueue.submitTask(task.getDescription(), task, null); // TODO should pass in request.masterNodeTimeout() here x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/CrossClusterQueryWithFiltersIT.java (5 lines): - line 190: // FIXME: this is currently inconsistent with the non-wildcard case, since empty wildcard is not an error, - line 280: // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime IndexNotFoundException - line 284: // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime IndexNotFoundException - line 323: // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime RemoteException - line 330: // FIXME: planner does not catch this now, it should be VerificationException but for now it's runtime RemoteException x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/termsenum/action/TransportTermsEnumAction.java (5 lines): - line 157: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can - line 422: // TODO remove this so we can shift code to server module - write a separate Interceptor class to - line 559: // TODO or remove above try and instead just call finishHim() here? Helps keep return logic consistent - line 691: // TODO: Handle exceptions in the atomic response array - line 699: // TODO: Handle exceptions in the atomic response array x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/AbstractPhysicalOperationProviders.java (5 lines): - line 128: // TODO: This is likely required only for pre-8.14 node compatibility; confirm and remove if possible. - line 137: // TODO: investigate whether a break could be used since it shouldn't be possible to have multiple - line 220: // TODO: This should take CATEGORIZE into account: - line 251: // TODO: investigate whether a break could be used since it shouldn't be possible to have multiple - line 287: // TODO: this needs to be made more reliable - use casting to blow up when dealing with expressions (e+1) x-pack/plugin/shutdown/src/internalClusterTest/java/org/elasticsearch/xpack/shutdown/NodeShutdownShardsIT.java (5 lines): - line 129: final String nodeB = "node_t1"; // TODO: fix this to so it's actually overrideable - line 185: final String nodeB = "node_t2"; // TODO: fix this to so it's actually overrideable - line 243: final String nodeB = "node_t1"; // TODO: fix this to so it's actually overrideable - line 262: final String nodeB = "node_t1"; // TODO: fix this to so it's actually overrideable - line 263: final String nodeC = "node_t2"; // TODO: fix this to so it's actually overrideable x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/ParseTables.java (5 lines): - line 44: // TODO use a real block factory - line 128: try (BytesRefBlock.Builder builder = blockFactory.newBytesRefBlockBuilder(100)) { // TODO 100?! - line 174: try (IntBlock.Builder builder = blockFactory.newIntBlockBuilder(100)) { // TODO 100?! - line 216: try (LongBlock.Builder builder = blockFactory.newLongBlockBuilder(100)) { // TODO 100?! - line 258: try (DoubleBlock.Builder builder = blockFactory.newDoubleBlockBuilder(100)) { // TODO 100?! x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/search/aggregations/bucket/geogrid/GeoHexGridTiler.java (5 lines): - line 53: // TODO: maybe we should remove this method from the API - line 67: // TODO: specialize when they are neighbour cells. - line 152: // TODO: specialize when they are neighbour cells. - line 226: // TODO: Verify this (and perhaps move the calculation into H3 and based on NUM_BASE_CELLS and others) - line 308: // TODO: Calculate correctly based on bounds x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/Security.java (5 lines): - line 493: // TODO: ip filtering does not actually track license usage yet - line 663: // TODO this is wrong, we should only use the environment that is provided to createComponents - line 1320: // TODO : The migration task needs to be project aware - line 1355: TimeValue.THIRTY_SECONDS /* TODO should this be configurable? longer by default? infinite? */, - line 2422: // TODO add bulkPutRoleRequestBuilderFactory loading here when available server/src/main/java/org/elasticsearch/index/IndexSettings.java (5 lines): - line 294: ); // TODO: remove setting - line 295: public static TimeValue STATELESS_DEFAULT_REFRESH_INTERVAL = TimeValue.timeValueSeconds(5); // TODO: this value is still not final - line 459: * TODO: Remove this setting in 9.0. - line 473: * TODO: Remove this setting in 9.0. - line 781: * TODO: Remove in 9.0 x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/FSTEnum.java (5 lines): - line 119: // TODO: should we return a status here (SEEK_FOUND / SEEK_NOT_FOUND / - line 127: // TODO: possibly caller could/should provide common - line 304: // TODO: possibly caller could/should provide common - line 506: // TODO: if each arc could somehow read the arc just - line 549: // TODO: possibly caller could/should provide common server/src/main/java/org/elasticsearch/index/codec/vectors/es816/ES816BinaryFlatVectorsScorer.java (5 lines): - line 64: // FIXME: precompute this once? - line 114: // FIXME: precompute this once? - line 140: // FIXME: pre-compute these only once for each target vector - line 177: // FIXME: pre-compute these only once for each target vector - line 184: // TODO maybe store? server/src/main/resources/org/elasticsearch/bootstrap/security.policy (5 lines): - line 104: // TODO: clean all these property writes up, and don't allow any more in. these are all bogus! - line 107: // TODO: put these in doPrivileged and move these to test-framework.policy - line 118: // TODO: set this with gradle or some other way that repros with seed? - line 121: // TODO: these simply trigger a noisy warning if its unable to clear the properties - line 136: // (TODO: clean this up?) x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/AuthenticatorChain.java (5 lines): - line 250: // TODO: In theory, handleNullToken should not be called at all if any credentials is extracted - line 255: // TODO: assert we really haven't extract any token - line 274: // TODO: we can also support run-as for fallback users if needed - line 275: // TODO: the authentication for fallback user is now serialised in the inner threadContext - line 308: // TODO: these should be different log messages if the runas vs auth user is disabled? x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/Util.java (5 lines): - line 49: // TODO: would be nice not to alloc this on every lookup - line 70: // TODO: maybe a CharsRef version for BYTE2 - line 78: // TODO: would be nice not to alloc this on every lookup - line 281: // TODO: we could enable FST to sorting arcs by weight - line 285: // TODO: maybe we should make an FST.INPUT_TYPE.BYTE0.5!? server/src/main/java/org/elasticsearch/index/codec/postings/ES812PostingsWriter.java (4 lines): - line 167: // TODO: should we try skipping every 2/4 blocks...? - line 349: // TODO: wasteful we are counting this (counting # docs - line 389: // TODO: should we send offsets/payloads to - line 501: // TODO: add a finish() at least to PushBase? DV too...? server/src/main/java/org/elasticsearch/cluster/metadata/Metadata.java (4 lines): - line 92: // TODO multi-project: verify that usages are really expected to work on the default project only, - line 284: * TODO: Revisit as part of multi-project (do we need it for BWC APIs / transport versions, or can we remove it) - line 460: * TODO: Remove as part of multi-project - line 1504: * TODO: This should map to {@link ProjectMetadata} (not Builder), but that's tricky to do due to server/src/main/java/org/elasticsearch/search/SearchService.java (4 lines): - line 708: // TODO: i think it makes sense to always do a canMatch here and - line 738: // TODO: this logic should be improved, the timeout should be handled in a way that removes the listener from the logic in the - line 1687: // TODO delay this construction until the FetchPhase is executed so that we can - line 1892: // TODO remove the exception handling as it's now in canMatch itself x-pack/plugin/searchable-snapshots/src/internalClusterTest/java/org/elasticsearch/xpack/searchablesnapshots/FrozenSearchableSnapshotsIntegTests.java (4 lines): - line 97: // we can bypass this by forcing soft deletes to be used. TODO this restriction can be lifted when #55142 is resolved. - line 286: // TODO: fix - line 342: // TODO: fix - line 371: // TODO: fix x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/LogicalPlanBuilder.java (4 lines): - line 177: String pattern = visitString(ctx.string()).fold(FoldContext.small() /* TODO remove me */).toString(); - line 208: String pattern = visitString(ctx.string()).fold(FoldContext.small() /* TODO remove me */).toString(); - line 263: result.put(visitIdentifier(option.identifier()), expression(option.constant()).fold(FoldContext.small() /* TODO remove me */)); - line 374: // TODO: add support for filters x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene40/blocktree/IntersectTermsEnumFrame.java (4 lines): - line 261: // TODO: maybe add scanToLabel; should give perf boost - line 310: // TODO: better API would be "jump straight to term=N"??? - line 313: // TODO: we could make "tiers" of metadata, ie, - line 319: // TODO: if docFreq were bulk decoded we could x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/RBACEngine.java (4 lines): - line 183: // TODO: When we expand support of workflows restriction to broader use cases (other than API keys for Search Application), - line 475: // TODO this may need to change. See https://github.com/elastic/elasticsearch/issues/105598 - line 928: // TODO: can this be done smarter? I think there are usually more indices/aliases in the cluster then indices defined a roles? - line 962: // TODO handle time checking in a follow-up server/src/internalClusterTest/java/org/elasticsearch/snapshots/SystemResourceSnapshotIT.java (4 lines): - line 1150: .indexPatterns(List.of(SYSTEM_DATASTREAM_NAME)) // TODO is this correct? - line 1190: .indexPatterns(List.of(SYSTEM_DATASTREAM_NAME)) // TODO is this correct? - line 1230: .indexPatterns(List.of(SYSTEM_DATASTREAM_NAME)) // TODO is this correct? - line 1299: .indexPatterns(List.of(SYSTEM_DATASTREAM_NAME)) // TODO is this correct? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/planner/QueryFolder.java (4 lines): - line 251: // TODO: remove exceptions from the Folder - line 277: // TODO: when dealing with expressions inside Aggregation, make sure to extract the field - line 706: // FIXME: concern leak - hack around MatrixAgg which is not - line 757: // TODO: might need to validate whether the target field or group actually exist server/src/main/java/org/elasticsearch/index/mapper/MapperService.java (4 lines): - line 581: // TODO: In many cases the source here is equal to mappingSource so we need not serialize again. - line 589: // TODO: can we even have concurrent updates here? - line 787: * TODO: remove in v 9.0 - line 828: // TODO this should bust the cache somehow. Tracked in https://github.com/elastic/elasticsearch/issues/66722 server/src/main/java/org/elasticsearch/index/codec/vectors/KMeans.java (4 lines): - line 173: // TODO: replace with RandomVectorScorer::score possible on quantized vectors - line 220: // TODO can we make this off-heap, or reusable? This could be a big array - line 323: // TODO: replace with RandomVectorScorer::score possible on quantized vectors - line 370: // TODO: replace with RandomVectorScorer::score possible on quantized vectors server/src/main/java/org/elasticsearch/index/IndexingPressure.java (4 lines): - line 36: // TODO: Remove once it is no longer needed for BWC - line 192: // TODO: Implement integration with IndexingPressure for unparsed bytes - line 204: // TODO: Eventually most of the memory will already be accounted for in unparsed. - line 252: // TODO: Maybe this should be re-named and used for primary operations too. Eventually we will need to account for: ingest pipeline x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/jwt/JwtRealm.java (4 lines): - line 115: // TODO: log invalid cache settings - line 273: // TODO: No point to continue to another realm if failure is ParseException - line 325: // TODO: evict the entry - line 432: // TODO: We should let the error bubble up instead of swallowing it server/src/main/java/org/elasticsearch/index/store/Store.java (4 lines): - line 128: * TODO: Remove in 9.0 - line 227: // TODO this should be caught by lucene - EOF is almost certainly an index corruption - line 690: // TODO do we need to also fail this if we can't delete the pending commit file? - line 878: // TODO we should check the checksum in lucene if we hit an exception x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/index/IndexResolver.java (4 lines): - line 316: // TODO: perform two requests w/ & w/o frozen option to retrieve (by diff) the throttling status? - line 319: // detail is missing, it's going to be a local index. TODO: why would this happen? - line 631: // TODO: to check whether isSearchable/isAggregateable takes into account the presence of the normalizer - line 796: // TODO is split still needed? x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/RollupResponseTranslator.java (4 lines): - line 307: // TODO it looks like this passes the "final" reduce context more than once. - line 461: // TODO expose getFormatter(), keyed upstream in Core - line 468: // TODO expose getFormatter(), keyed upstream in Core - line 582: // TODO comment from Colin in review: server/src/main/java/org/elasticsearch/cluster/coordination/Coordinator.java (4 lines): - line 357: // TODO this is a workaround for log4j's Supplier. We should remove this, once using ES logging api - line 943: // TODO tests for heartbeat failures - line 1414: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 1469: // TODO: add test coverage for this branch server/src/main/java/org/elasticsearch/ElasticsearchException.java (4 lines): - line 412: // TODO: we could walk the exception chain to see if _any_ causes are timeouts? - line 579: // TODO move subclasses that print out simple metadata to using addMetadata directly and support also numbers and booleans. - line 580: // TODO rename metadataToXContent and have only SearchPhaseExecutionException use it, which prints out complex objects - line 767: // TODO: do we really need to make the exception name in underscore casing? x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/AutoFollowCoordinator.java (4 lines): - line 268: // TODO: set non-compliant status on auto-follow coordination that can be viewed via a stats API - line 300: // TODO: set non-compliant status on auto-follow coordination that can be viewed via a stats API - line 388: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 716: // TODO use longer timeouts here? see https://github.com/elastic/elasticsearch/issues/109150 x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/join/Join.java (4 lines): - line 184: // TODO: make the other side nullable - line 204: * TODO we should rework stats so we don't have to do this - line 276: // TODO: Add support for VERSION by implementing QueryList.versionTermQueryList similar to ipTermQueryList - line 286: // TODO: Consider allowing more valid types x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/xpack/searchablesnapshots/allocation/SearchableSnapshotAllocator.java (4 lines): - line 137: // TODO: cancel and jump to better available allocations? - line 275: // TODO: in the following logic, we do not account for existing cache size when handling disk space checks, should and can we - line 306: // TODO: does this make sense? Unlike with the store we could evict the cache concurrently and wait for nothing? - line 329: // TODO: do we need handling of delayed allocation for leaving replicas here? x-pack/plugin/text-structure/src/main/java/org/elasticsearch/xpack/textstructure/structurefinder/GrokPatternCreator.java (4 lines): - line 119: // TODO: would be nice to have IPORHOST here, but HOSTNAME matches almost all words - line 138: // TODO: also unfortunately can't have USERNAME in the list as it matches too broadly - line 172: // TODO: would be nice to have IPORHOST here, but HOSTNAME matches almost all words - line 184: // TODO: also unfortunately can't have USERNAME in the list as it matches too broadly server/src/main/java/org/elasticsearch/repositories/RepositoriesService.java (4 lines): - line 233: // TODO contemplate threading, do we need to fork, see #101445? - line 395: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 653: // TODO: this catch is bogus, it means the old repo is already closed, - line 698: EsExecutors.DIRECT_EXECUTOR_SERVICE, // TODO contemplate threading here, do we need to fork, see #101445? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanArrayVector.java (4 lines): - line 32: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 34: // TODO: remove this if/when we account for memory used by Pages - line 65: // TODO: One bit for each boolean - line 115: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/common/InferenceServiceNodeLocalRateLimitCalculator.java (4 lines): - line 65: // TODO: should probably be a map/set - line 95: // TODO: feature flag per node? We should not reroute to nodes not having eis and/or the inference plugin enabled - line 172: // TODO: here we can probably be smarter: if |num nodes in cluster| > |num nodes per task types| - line 190: // TODO: this needs to take in service and task type as soon as multiple services/task types are supported server/src/main/java/org/elasticsearch/action/update/UpdateHelper.java (4 lines): - line 63: // TODO: Don't hard-code gFields - line 335: * // TODO can we pass a Source here rather than Map, XcontentType and BytesReference? - line 357: // TODO when using delete/none, we can still return the source as bytes by generating it (using the sourceContentType) - line 436: // TODO: can we remove this leniency yet?? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/Block.java (4 lines): - line 46: * TODO maybe make this everywhere? - line 86: // TODO: renaming this to owning blockFactory once we pass blockFactory for filter and expand - line 140: * TODO: pass BlockFactory - line 231: // TODO remove default and scatter to implementation where it can be a lot more efficient modules/repository-s3/src/main/java/org/elasticsearch/repositories/s3/S3BlobContainer.java (4 lines): - line 996: // TODO should we sort these by initiation time (and then upload ID as a tiebreaker)? - line 997: // TODO should we listMultipartUploads() while waiting, so we can fail quicker if we are concurrently cancelled? - line 1155: // TODO adjust to use S3BlobStore.configureRequestForMetrics, adding metrics collection - line 1186: // TODO adjust to use S3BlobStore.configureRequestForMetrics, adding metrics collection x-pack/plugin/sql/jdbc/src/main/java/org/elasticsearch/xpack/sql/jdbc/JdbcResultSet.java (4 lines): - line 70: // TODO: should we consider the locale as well? - line 255: // TODO: the error message in case the value in the column cannot be converted to a Date refers to a column index - line 270: // TODO: the B6 appendix of the jdbc spec does mention CHAR, VARCHAR, LONGVARCHAR, DATE, TIMESTAMP as supported - line 276: // TODO: this should probably be handled server side x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/X-ArrayVector.java.st (4 lines): - line 35: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 37: // TODO: remove this if/when we account for memory used by Pages - line 103: // TODO: One bit for each boolean - line 183: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvMedian.java (4 lines): - line 118: // TODO quickselect - line 148: // TODO quickselect - line 186: // TODO quickselect - line 222: // TODO quickselect x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/IndexResolver.java (4 lines): - line 106: // TODO flattened is simpler - could we get away with that? - line 133: // TODO we're careful to make isAlias match IndexResolver - but do we use it? - line 223: // TODO I think we only care about unmapped fields if we're aggregating on them. do we even then? - line 230: // TODO: to check whether isSearchable/isAggregateable takes into account the presence of the normalizer server/src/main/java/org/elasticsearch/index/shard/IndexShard.java (4 lines): - line 1082: // TODO this can be removed if we eagerly create mappings as soon as a new index is created, regardless of - line 2138: // TODO: Don't enable this leniency unless users explicitly opt-in - line 3486: // TODO: Create a proper object to encapsulate the recovery context - line 4598: // TODO: add a dedicate recovery stats for the reset translog x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene40/blocktree/IntersectTermsEnum.java (4 lines): - line 71: // TODO: in some cases we can filter by length? eg - line 95: // TODO: if the automaton is "smallish" we really - line 185: // TODO: we could be more efficient for the next() - line 484: // TODO: maybe we should do the same linear test x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/mvdedupe/BatchEncoder.java (4 lines): - line 331: // TODO some protection against growing to gigabytes or whatever - line 399: // TODO some protection against growing to gigabytes or whatever - line 467: // TODO some protection against growing to gigabytes or whatever - line 593: // TODO some protection against growing to gigabytes or whatever x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/predicate/operator/comparison/In.java (4 lines): - line 220: // TODO: update this null check in QL too? - line 246: protected TypeResolution resolveType() { // TODO: move the foldability check from QL's In to SQL's and remove this method - line 484: // TODO: - line 494: terms.add(valueOf(FoldContext.small() /* TODO remove me */, rhs)); server/src/main/java/org/elasticsearch/action/search/SearchQueryThenFetchAsyncAction.java (4 lines): - line 586: // TODO: log activating or otherwise limiting parallelism might be helpful here - line 700: // TODO: count down fully and just respond with an exception if partial results aren't allowed as an - line 708: // TODO this could be done better now, we probably should only make sure to have a single loop running at - line 848: // TODO: dry up the bottom sort collector with the coordinator side logic in the top-level class here x-pack/plugin/security/cli/src/main/java/org/elasticsearch/xpack/security/cli/HttpCertificateCommand.java (3 lines): - line 345: // TODO : Should we add support for configuring PKI in ES? - line 350: // TODO Add info to the READMEs so that the user could regenerate these certs if needed. - line 453: // TODO : Should we add support for client certs from Kibana to ES? libs/entitlement/src/main/java/org/elasticsearch/entitlement/runtime/policy/FileAccessTree.java (3 lines): - line 259: // TODO: null paths shouldn't be allowed, but they can occur due to repo paths - line 268: // TODO: this grants read access to the config dir for all modules until explicit read entitlements can be added - line 274: // TODO: watcher uses javax.activation which looks for known mime types configuration, should this be global or explicit in watcher? server/src/main/java/org/elasticsearch/search/sort/GeoDistanceSortBuilder.java (3 lines): - line 537: // TODO implement the single point optimization above - line 580: // TODO this lines up with FieldSortBuilder. Share? - line 606: // TODO this is pretty similar to FieldSortBuilder. Share? server/src/main/java/org/elasticsearch/lucene/grouping/SinglePassGroupingCollector.java (3 lines): - line 58: * TODO: If the sort is based on score we should propagate the mininum competitive score when orderedGroups - line 299: // TODO: should we add option to mean "ignore docs that - line 386: // TODO: optimize this server/src/main/java/org/elasticsearch/indices/cluster/IndicesClusterStateService.java (3 lines): - line 299: // TODO: feels hacky, a block disables state persistence, and then we clean the allocated shards, maybe another flag in blocks? - line 348: if (masterNode != null) { // TODO: can we remove this? Is resending shard failures the responsibility of shardStateAction? - line 811: // TODO could we instead subscribe to the shard lock and trigger the retry exactly when it is released rather than polling? server/src/main/java/org/elasticsearch/action/search/CanMatchPreFilterSearchPhase.java (3 lines): - line 185: // TODO: the index filter (i.e, `_index:patten`) should be prefiltered on the coordinator - line 433: // TODO: pick the local shard when possible - line 496: // TODO: we could fail the request because there is a high probability x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/mapper/Mapper.java (3 lines): - line 91: // TODO - this is hard to follow and needs reworking - line 152: // TODO: might be easier long term to end up with just one node and split if necessary instead of doing that always at this - line 191: // TODO: share code with local LocalMapper? x-pack/plugin/text-structure/src/main/java/org/elasticsearch/xpack/textstructure/structurefinder/TimestampFormatFinder.java (3 lines): - line 585: // TODO consider support for overriding the locale too - line 1663: // TODO consider support for overriding the locale too - line 1675: // TODO consider support for overriding the locale too server/src/main/java/org/elasticsearch/index/codec/postings/ES812PostingsReader.java (3 lines): - line 862: // TODO: in theory we could avoid loading frq block - line 1384: // TODO: in theory we could avoid loading frq block - line 1806: // TODO: in theory we could avoid loading frq block server/src/main/java/org/elasticsearch/repositories/RepositoryData.java (3 lines): - line 360: // TODO this method is pretty opaque, let's add some comments - line 730: // by earlier versions, but still not great. TODO rethink this if and when adding a new snapshot repository format version. - line 1165: @Nullable // TODO forbid nulls here, this only applies to very old repositories build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/precommit/DependencyLicensesTask.java (3 lines): - line 103: // TODO: we should be able to default this to eg compile deps, but we need to move the licenses - line 226: // TODO: why do we support suffix of LICENSE *and* LICENSE.txt?? - line 312: // try the other suffix...TODO: get rid of this, just support ending in .txt x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/expression/predicate/operator/math/Maths.java (3 lines): - line 57: int digitsToRound = safeToInt(-precision); // TODO: why is precision a long? - line 60: BigInteger middleResult = n.divide(tenAtScale); // TODO: "intermediateResult"? - line 86: long remainder = n % tenAtScale; // TODO: vs.: n - middleResult * tenAtScale modules/legacy-geo/src/main/java/org/elasticsearch/legacygeo/builders/PolygonBuilder.java (3 lines): - line 562: // TODO Can this actually happen? Needs a test to exercise it, or else needs to be removed. - line 569: index = 0; // TODO Should this be pos instead of 0? This assigns exact matches to the southernmost component. - line 608: // TODO: Check if we could save the set null step x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/XPackPlugin.java (3 lines): - line 147: // TODO: clean up this library to not ask for write access to all system properties! - line 167: // TODO: fix gradle to add all security resources (plugin metadata) to test classpath - line 188: // FIXME: The settings might be changed after this (e.g. from "additionalSettings" method in other plugins) x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene40/blocktree/SegmentTermsEnum.java (3 lines): - line 381: // TODO: reverse vLong byte order for better FST - line 656: // TODO: we should write our vLong backwards (MSB - line 674: // TODO: we could save the outputs in local server/src/main/java/org/elasticsearch/common/util/concurrent/EsExecutors.java (3 lines): - line 329: // TODO this should only be allowed in tests - line 335: // TODO missing node names should only be allowed in tests - line 580: // This is a random starting point alpha. TODO: revisit this with actual testing and/or make it configurable modules/aggregations/src/main/java/org/elasticsearch/aggregations/bucket/histogram/AutoDateHistogramAggregator.java (3 lines): - line 102: // TODO: Remove null usage here, by using a different aggregator for create - line 432: private long nextRebucketAt = 1000; // TODO this could almost certainly start higher when asMultiBucketAggregator is gone - line 611: * TODO it'd be faster if we could apply the merging on the fly as we x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/conditional/Case.java (3 lines): - line 231: if (Boolean.TRUE.equals(condition.condition.fold(FoldContext.small() /* TODO remove me - use literal true?*/))) { - line 269: // TODO don’t throw away the results of any `fold`. That might mean looking for literal TRUE on the conditions. - line 527: // TODO Copy the per-type specialization that COALESCE has. server/src/main/java/org/elasticsearch/index/IndexService.java (3 lines): - line 468: * TODO: we execute this in parallel but it's a synced method. Yet, we might - line 507: // TODO: we should, instead, hold a "bytes reserved" of how large we anticipate this shard will be, e.g. for a shard - line 1393: // TODO add specific tests to SearchExecutionTests similar to the ones in FieldTypeLookupTests x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/physical/local/PushStatsToSource.java (3 lines): - line 58: // TODO: handle case where some aggs cannot be pushed down by breaking the aggs into two sources (regular + stats) + union - line 91: // TODO: add count over field (has to be field attribute) - line 131: // TODO: the attributes have been recreated here; they will have wrong name ids, and the dependency check will server/src/main/java/org/elasticsearch/action/search/SearchPhaseController.java (3 lines): - line 314: // TODO it would be nice to assert this in the future - line 367: // TODO it would be nice to assert this in the future - line 545: * //TODO: instead of throwing error, find a way to sort long and unsigned_long together x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/PushDownAndCombineFilters.java (3 lines): - line 44: // TODO: Push down past STATS if the filter is only on the groups; but take into account how `STATS ... BY field` handles - line 89: // TODO: could we do better here about pushing down filters for inlinestats? - line 122: // TODO: generalize this for other join types server/src/main/java/org/elasticsearch/search/rescore/QueryRescorer.java (3 lines): - line 57: // TODO: shouldn't this be up to the ScoreMode? I.e., we should just invoke ScoreMode.combine, passing 0.0f for the - line 147: // TODO: shouldn't this be up to the ScoreMode? I.e., we should just invoke ScoreMode.combine, passing 0.0f for the - line 152: // TODO: this is wrong, i.e. we are comparing apples and oranges at this point. It would be better if we always rescored all x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/LookupJoinExec.java (3 lines): - line 111: // TODO: this is a hack since the right side is always materialized - instead this should - line 118: // TODO: same as above - once lazy materialization of both sides lands, this needs updating - line 129: // TODO: currently it's hard coded that we add all fields from the lookup index. But the output we "officially" get from the right server/src/main/java/org/elasticsearch/action/fieldcaps/TransportFieldCapabilitiesAction.java (3 lines): - line 105: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 138: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can - line 307: // TODO: remove this workaround after we fixed https://github.com/elastic/elasticsearch/issues/107439 x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/enrich/AbstractLookupService.java (3 lines): - line 232: // TODO: handle retry and avoid forking for the local lookup - line 284: if (mergePages // TODO fix this optimization for Lookup. - line 500: // TODO: Remove this workaround once we have Block RefCount x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/blockhash/BytesRefBlockHash.java (3 lines): - line 55: // TODO track raw counts and which implementation we pick for the profiler - #114008 - line 112: // TODO honor targetBlockSize and chunk the pages if requested. - line 194: // TODO replace with takeBytesRefsOwnership ?! server/src/main/java/org/elasticsearch/common/settings/Setting.java (3 lines): - line 172: * TODO Should be checked in {@link Setting#isDeprecatedAndRemoved} - line 1233: // TODO should we be checking for deprecations here? - line 1865: // TODO this one's two argument get is still broken x-pack/plugin/ent-search/src/main/java/org/elasticsearch/xpack/application/analytics/AnalyticsCollectionService.java (3 lines): - line 90: TimeValue.THIRTY_SECONDS /* TODO should we wait longer? */, - line 91: TimeValue.THIRTY_SECONDS /* TODO should we wait longer? */, - line 137: TimeValue.THIRTY_SECONDS /* TODO should we wait longer? */, x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/LocalExecutionPlanner.java (3 lines): - line 699: // TODO support multiple match fields, and support more than equality predicates - line 761: // TODO: should this be extracted into a separate eval block? - line 794: int blockSize = 5000;// TODO estimate row size and use context.pageSize() server/src/main/java/org/elasticsearch/plugins/PluginsUtils.java (3 lines): - line 294: // TODO: optimize to skip any bundles not connected to the candidate plugin? - line 300: // TODO: no jars should be an error - line 301: // TODO: verify the classname exists in one of the jars! x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/XPackClientPlugin.java (3 lines): - line 120: // TODO: merge this into XPackPlugin - line 129: // TODO split these settings up - line 419: // TODO: The WeightedTokensBuilder is slated for removal after the SparseVectorQueryBuilder is available. server/src/main/java/org/elasticsearch/action/search/TransportSearchAction.java (3 lines): - line 453: // TODO: pass parentTaskId - line 851: // TODO: support point-in-time - line 1229: // TODO: I think startTime() should become part of ActionRequest and that should be used both for index name server/src/main/java/org/elasticsearch/common/geo/GeoPolygonDecomposer.java (3 lines): - line 273: // TODO: Check if we could save the set null step - line 511: // TODO Can this actually happen? Needs a test to exercise it, or else needs to be removed. - line 518: index = 0; // TODO Should this be pos instead of 0? This assigns exact matches to the southernmost component. x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntArrayVector.java (3 lines): - line 32: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 34: // TODO: remove this if/when we account for memory used by Pages - line 124: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. modules/lang-painless/src/main/java/org/elasticsearch/painless/MethodWriter.java (3 lines): - line 107: // TODO: introduce a way to ignore internal statements so this assert is not triggered - line 108: // TODO: https://github.com/elastic/elasticsearch/issues/51836 - line 119: // TODO: maybe track these in bitsets too? this is trickier... server/src/main/java/org/elasticsearch/common/util/CuckooFilter.java (3 lines): - line 110: // TODO this is probably super slow, but just used for testing atm - line 310: // TODO implement semi-sorting - line 451: * TODO: there are schemes to avoid powers of two, might want to investigate those x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/assignment/TrainedModelAssignmentClusterService.java (3 lines): - line 159: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 800: // TODO This checks if there is more space we could vertically scale to but - line 1181: // TODO this has a weird side-effect for allocating to nodes server/src/main/java/org/elasticsearch/transport/TransportService.java (3 lines): - line 387: /* other transports (used in tests) may not implement the proper close-connection behaviour. TODO fix this. */ - line 952: // TODO we can probably fold this entire request ID dance into connection.sendRequest but it will be a bigger refactoring - line 1173: // TODO we should makes this a hard validation and throw an exception but we need a good way to add backwards layer x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvMedianAbsoluteDeviation.java (3 lines): - line 217: // TODO quickselect - line 281: // TODO quickselect - line 344: // TODO quickselect build-tools/src/main/java/org/elasticsearch/gradle/testclusters/ElasticsearchNode.java (3 lines): - line 256: // TODO Refactor test using register<> for reducing overhead - line 554: // TODO revisit when moving to shorter paths on CI by using Teamcity - line 615: * //TODO: Remove this when system modules are available server/src/main/java/org/elasticsearch/cluster/metadata/ProjectMetadata.java (3 lines): - line 845: // TODO: This can be moved to IndexNameExpressionResolver too, but this means that we will support wildcards and other expressions - line 1602: // TODO: We should move these datastructures to IndexNameExpressionResolver, this will give the following benefits: - line 1683: // TODO: I think we can remove these arrays. it isn't worth the effort, for operations on all indices. server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/BalancedShardsAllocator.java (3 lines): - line 625: * TODO we could be a bit smarter here, we don't need to fully sort necessarily - line 820: // TODO maybe we can respect throttling here too? - line 895: * TODO: We could be smarter here and group the shards by index and then x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/TransformIndexer.java (3 lines): - line 1128: .indicesOptions(IndicesOptions.LENIENT_EXPAND_OPEN); // TODO: make configurable - line 1138: // TODO: if buildChangesQuery changes the query it get overwritten - line 1191: .indicesOptions(IndicesOptions.LENIENT_EXPAND_OPEN); // TODO: make configurable x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/X-BlockHash.java.st (3 lines): - line 85: // TODO track raw counts and which implementation we pick for the profiler - #114008 - line 152: // TODO honor targetBlockSize and chunk the pages if requested. - line 245: // TODO replace with takeBytesRefsOwnership ?! x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleArrayVector.java (3 lines): - line 32: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 34: // TODO: remove this if/when we account for memory used by Pages - line 114: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/blob-cache/src/main/java/org/elasticsearch/blobcache/shared/SharedBytes.java (3 lines): - line 64: // TODO: for systems like Windows without true p-write/read support we should split this up into multiple channels since positional - line 129: // TODO: be resilient to this check failing and try next path? - line 135: // TODO: leave some margin for error here x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/permission/IndicesPermission.java (3 lines): - line 235: // TODO remove me (this has >700 usages in tests which would make for a horrible diff; will remove this once the main PR is merged) - line 386: // TODO The `false` result is a _safe_ default but this is actually an error. Make it an error. - line 912: // TODO: Use FieldPermissionsDefinition instead of FieldPermissions. The former is a better counterpart to query server/src/main/java/org/elasticsearch/search/query/QueryPhaseCollector.java (3 lines): - line 102: // TODO for aggs that return TOP_DOCS, score mode becomes exhaustive unless top docs collector agrees on the score mode - line 105: // TODO if we had TOP_DOCS, shouldn't we return TOP_DOCS_WITH_SCORES instead of COMPLETE? - line 321: // TODO we expose the competitive iterator only when one of the two sub-leaf collectors has early terminated, x-pack/plugin/ml-package-loader/src/main/java/org/elasticsearch/xpack/ml/packageloader/action/TransportGetTrainedModelPackageConfigAction.java (3 lines): - line 106: // TODO: use proper ElasticsearchStatusExceptions - line 110: // TODO: what if the URI contained credentials, don't leak it in the exception - line 113: // TODO: don't leak the full url and package details x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/RemoteClusterSecurityEsqlIT.java (3 lines): - line 585: // TODO: in follow on PR, figure out why this is returning the wrong error - should be "cannot connect to invalid_remote" - line 623: // TODO: in follow on PR, figure out why this is returning the wrong error - should be "cannot connect to invalid_remote" - line 1221: // TODO: in follow on PR, add support for throwing a VerificationException for this scenario - no exception is currently thrown modules/lang-painless/src/main/java/org/elasticsearch/painless/AnalyzerCaster.java (3 lines): - line 563: // TODO: In the rare case we still haven't reached a correct promotion we need - line 564: // TODO: to calculate the highest upper bound for the two types and return that. - line 565: // TODO: However, for now we just return objectType that may require an extra cast. x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BytesRefArrayVector.java (3 lines): - line 33: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 35: // TODO: remove this if/when we account for memory used by Pages - line 115: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/predicate/operator/math/Maths.java (3 lines): - line 57: int digitsToRound = safeToInt(-precision); // TODO: why is precision a long? - line 60: BigInteger middleResult = n.divide(tenAtScale); // TODO: "intermediateResult"? - line 86: long remainder = n % tenAtScale; // TODO: vs.: n - middleResult * tenAtScale x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/AggregateMetricDoubleBlock.java (3 lines): - line 199: // TODO: support - line 205: // TODO: determine based on sub-blocks - line 211: // TODO: support x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/ExpressionBuilder.java (3 lines): - line 545: // TODO we could handle this a bit better (like ES SQL does it) so that -(-(-123)) results in the -123 the Literal - line 752: new WildcardPattern(pattern.fold(FoldContext.small() /* TODO remove me */).toString()) - line 761: new RLikePattern(pattern.fold(FoldContext.small() /* TODO remove me */).toString()) x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/TransportEsqlQueryAction.java (3 lines): - line 108: // TODO replace SAME when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 177: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can - line 215: // TODO: plug-in security server/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/get/SnapshotSortKey.java (3 lines): - line 58: // TODO: cover via pre-flight predicate - line 93: // TODO: cover via pre-flight predicate - line 139: // TODO: cover via pre-flight predicate x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/FloatArrayVector.java (3 lines): - line 32: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 34: // TODO: remove this if/when we account for memory used by Pages - line 114: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. modules/lang-expression/src/main/java/org/elasticsearch/script/expression/ExpressionScriptEngine.java (3 lines): - line 293: // TODO: if some uses it in a scoring expression, they will get a nasty failure when evaluating...need a - line 364: // TODO: if some uses it in a scoring expression, they will get a nasty failure when evaluating...need a - line 489: // TODO: document and/or error if params contains _score? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongArrayVector.java (3 lines): - line 32: // TODO: remove these extra bytes once `asBlock` returns a block with a separate reference to the vector. - line 34: // TODO: remove this if/when we account for memory used by Pages - line 114: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/esql/arrow/src/main/java/org/elasticsearch/xpack/esql/arrow/ArrowResponse.java (3 lines): - line 387: // FIXME: counters: are they signed? - line 398: // FIXME: is it signed? - line 416: // TODO: support also CBOR and SMILE with an additional formatting parameter server/src/main/java/org/elasticsearch/action/support/replication/TransportWriteAction.java (3 lines): - line 446: // TODO: Temporary until we fail unpromotable shard - line 494: // TODO: Temporary until we fail unpromotable shard - line 519: // TODO: Maybe move this into PostWriteRefresh x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/IndexLifecycleService.java (3 lines): - line 190: // TODO multi-project: this probably needs a per-project iteration - line 655: // TODO: in the future we could take proactive measures for when a shutdown is actually triggered - line 658: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here modules/analysis-common/src/main/java/org/elasticsearch/analysis/common/CommonAnalysisPlugin.java (3 lines): - line 385: // TODO deprecate and remove in API - line 509: // TODO this one seems useless - line 599: // TODO deprecate and remove in API x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/IntBucketedSort.java (2 lines): - line 155: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 199: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/persistence/TrainedModelProvider.java (2 lines): - line 556: // TODO Change this when we get more than just langIdent stored - line 583: // TODO how could we stream in the model definition WHILE parsing it? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ilm/SearchableSnapshotAction.java (2 lines): - line 203: // TODO: allow this behavior instead of returning false, in this case the index is already a searchable a snapshot - line 310: // TODO: allow this behavior instead of throwing an exception server/src/internalClusterTest/java/org/elasticsearch/search/SearchCancellationIT.java (2 lines): - line 58: // TODO all tests need to be updated to work with concurrent search - line 243: // TODO: make this test compatible with batched execution, currently the exceptions are slightly different with batched libs/core/src/main/java/org/elasticsearch/core/IOUtils.java (2 lines): - line 210: // TODO: remove this leniency - line 258: // TODO: replace with constants class if needed (cf. org.apache.lucene.util.Constants) server/src/main/java/org/elasticsearch/common/network/NetworkService.java (2 lines): - line 154: // TODO: needs to be InetAddress[] - line 168: // TODO: allow publishing multiple addresses server/src/main/java/org/elasticsearch/persistent/PersistentTasksClusterService.java (2 lines): - line 202: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 807: // TODO just run on the elected master? x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene54/Lucene54DocValuesConsumer.java (2 lines): - line 136: // TODO: more efficient? - line 334: // TODO: in some cases representing missing with minValue-1 wouldn't take up additional space and so on, server/src/main/java/org/elasticsearch/search/sort/BucketedSort.java (2 lines): - line 186: // TODO we usually have a heap here so we could use that to build the results sorted. - line 376: // TODO a "bottom up" insert would save a couple of comparisons. Worth it? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/LastOverTimeDoubleAggregator.java (2 lines): - line 36: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 96: // TODO: handle multiple values? server/src/main/java/org/elasticsearch/index/engine/EngineConfig.java (2 lines): - line 130: * TODO: Remove in 9.0 - line 205: // TODO: Remove this escape hatch in 8.x x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportStopTrainedModelDeploymentAction.java (2 lines): - line 109: // TODO check for alias - line 110: // TODO model or deployment? x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/IndexedDISI.java (2 lines): - line 218: // TODO: binary search - line 234: // TODO: binary search server/src/main/java/org/elasticsearch/rest/RestController.java (2 lines): - line 404: // TODO consider refactoring to handler.supportsContentStream(xContentType). It is only used with JSON and SMILE - line 426: // TODO: estimate streamed content size for circuit breaker, x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/FilterOperator.java (2 lines): - line 49: // TODO we can detect constant true or false from the type - line 50: // TODO or we could make a new method in bool-valued evaluators that returns a list of numbers x-pack/plugin/esql/tools/src/main/java/org/elasticsearch/xpack/esql/tools/ProfileParser.java (2 lines): - line 178: // TODO: Sleeps have more details that could be added here - line 185: // TODO: Add status; needs standardizing the operatur statuses, maybe. build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/ElasticsearchJavaBasePlugin.java (2 lines): - line 152: // TODO Discuss moving compileOptions.getCompilerArgs() to use provider api with Gradle team. - line 168: // TODO: this probably shouldn't apply to groovy at all? server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/LongTermsIT.java (2 lines): - line 94: private static final int NUM_DOCS = 5; // TODO randomize the size? - line 117: IndexRequestBuilder[] highCardBuilders = new IndexRequestBuilder[100]; // TODO randomize the size? build-tools/src/main/java/org/elasticsearch/gradle/plugin/BasePluginBuildPlugin.java (2 lines): - line 85: // TODO: use explodedPlugin here for modules - line 199: bundleSpec.from("src/main/packaging");// TODO: move all config/bin/_size/etc into packaging x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/multiterms/MultiTermsAggregator.java (2 lines): - line 173: * TODO: this is a temporary solution, we should replace it with a more optimal mechanism instead of relying on BytesKeyedBucketOrds - line 207: // TODO: this is a temporary solution, we should replace it with a more optimal mechanism instead of relying on server/src/main/java/org/elasticsearch/cluster/metadata/MetadataDataStreamsService.java (2 lines): - line 281: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 556: // TODO: change order when "delete index api" also updates the data stream the "index to be removed" is a member of x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/logging/ThrottlerManager.java (2 lines): - line 30: * TODO remove in 10.0 - line 44: * TODO remove in 10.0 server/src/main/java/org/elasticsearch/lucene/search/uhighlight/CustomUnifiedHighlighter.java (2 lines): - line 274: * TODO: We should add the support for keyword runtime fields. - line 287: * TODO: We should add the support for keyword runtime fields. server/src/main/java/org/elasticsearch/action/support/broadcast/TransportBroadcastAction.java (2 lines): - line 69: // TODO replace SAME when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 91: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/persistence/JobConfigProvider.java (2 lines): - line 602: // TODO A better way to handle this rather than just ignoring the error? - line 730: // TODO A better way to handle this rather than just ignoring the error? server/src/main/java/org/elasticsearch/index/query/MoreLikeThisQueryBuilder.java (2 lines): - line 454: // TODO we allow null here for the _all field, but this is forbidden in the parser. Re-check - line 1125: // TODO this needs heavy cleanups before we can rewrite it x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/sort/IpBucketedSort.java (2 lines): - line 146: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 190: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/EsqlActionIT.java (2 lines): - line 114: // TODO: Allow relocation once we have retry in ESQL (see #103081) - line 377: // TODO: each aggregator returns Double now, it should in fact mirror the data type of the fields it's aggregating x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/job/persistence/AnomalyDetectorsIndex.java (2 lines): - line 110: // TODO: shard count default preserves the existing behaviour when the - line 146: // TODO: shard count default preserves the existing behaviour when the x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/sort/BytesRefBucketedSort.java (2 lines): - line 160: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 222: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/datafeed/DatafeedConfig.java (2 lines): - line 354: // TODO Remove in v8.0.0 - line 402: // TODO refactor in v8.0.0 x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/Watcher.java (2 lines): - line 834: WatcherServiceRequest serviceRequest = new WatcherServiceRequest(TimeValue.THIRTY_SECONDS /* TODO should this be longer? */); - line 855: WatcherServiceRequest serviceRequest = new WatcherServiceRequest(TimeValue.THIRTY_SECONDS /* TODO should this be longer? */); modules/aggregations/src/main/java/org/elasticsearch/aggregations/bucket/timeseries/InternalTimeSeries.java (2 lines): - line 40: // TODO: make computing docCount optional - line 199: List bucketsWithSameKey = new ArrayList<>(size); // TODO: not sure about this size? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/transform/TransformField.java (2 lines): - line 55: // TODO: Rename to "defer_data_validation" or similar to emphasize that not all validation is deferred - line 73: // TODO: consider a lower bound for usecases where you want to transform on a window of a stream x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/action/saml/TransportSamlLogoutAction.java (2 lines): - line 54: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 63: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can server/src/internalClusterTest/java/org/elasticsearch/snapshots/ConcurrentSnapshotsIT.java (2 lines): - line 1160: // TODO: remove this leniency once we fix the API to handle master failover cleaner - line 1174: // TODO: remove this leniency once we fix the API to handle master failover cleaner server/src/main/java/org/elasticsearch/lucene/spatial/ShapeDocValuesQuery.java (2 lines): - line 137: return 1000f; // TODO: what should it be? - line 198: return 1000f; // TODO: what should it be? server/src/main/java/org/elasticsearch/index/mapper/MappingParser.java (2 lines): - line 130: // TODO this should be the final value once `_source.mode` mapping parameter is not used anymore - line 183: * TODO: server/src/main/java/org/elasticsearch/common/xcontent/XContentHelper.java (2 lines): - line 127: // TODO now that we have config we make a method on bytes to do this building without needing this check everywhere - line 742: * TODO remove in ES v9 x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/search/aggregations/support/CartesianPointValuesSourceType.java (2 lines): - line 45: // TODO (support scripts) - line 66: // TODO: also support the structured formats of points x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/predicate/Range.java (2 lines): - line 228: Object l = valueOf(FoldContext.small() /* TODO remove me */, lower); - line 229: Object u = valueOf(FoldContext.small() /* TODO remove me */, upper); server/src/main/java/org/elasticsearch/index/mapper/FieldMapper.java (2 lines): - line 196: // TODO: multi fields are really just copy fields, we just need to expose "sub fields" or something that can be part - line 670: // TODO: multi fields are really just copy fields, we just need to expose "sub fields" or something that can be part x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/LastOverTimeFloatAggregator.java (2 lines): - line 36: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 96: // TODO: handle multiple values? x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/SnapshotRetentionTask.java (2 lines): - line 250: // TODO: Use snapshot multi-delete instead of this loop if all nodes in the cluster support it - line 333: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/PlannerUtils.java (2 lines): - line 87: // TODO: we are adding a Project plan to force InsertFieldExtraction - we should remove this transformation - line 313: * TODO: Remove this server/src/main/java/org/elasticsearch/search/aggregations/support/TimeSeriesIndexSearcher.java (2 lines): - line 46: * TODO: Convert it to use index sort instead of hard-coded tsid and timestamp values - line 214: private final SortedNumericDocValues timestamps; // TODO can we have this just a NumericDocValues? server/src/main/java/org/elasticsearch/index/engine/LuceneChangesSnapshot.java (2 lines): - line 252: // TODO: Callers should ask for the range that source should be retained. Thus we should always - line 263: // TODO: pass the latest timestamp from engine. x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/assignment/TrainedModelAssignmentNodeService.java (2 lines): - line 303: // TODO this means requests may still be routed here, should we not stop deployment? - line 794: // TODO: Do we want to stop the task? This would cause it to be reloaded by state updates on INITIALIZING x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/Warnings.java (2 lines): - line 34: // TODO: rename to createWarningsTreatedAsNull - line 66: // TODO: rename to createWarnings x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/generative/GenerativeRestTest.java (2 lines): - line 44: "must be \\[any type except counter types\\]", // TODO refine the generation of count() - line 48: "evaluation of \\[.*\\] failed, treating result as null", // TODO investigate? server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/DateHistogramIT.java (2 lines): - line 118: // TODO: would be nice to have more random data here - line 1399: // TODO: add some tests for negative fixed and calendar intervals x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleBigArrayBlock.java (2 lines): - line 27: private static final long BASE_RAM_BYTES_USED = 0; // TODO: fix this - line 132: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/IncludeExclude.java (2 lines): - line 60: // TODO: move all aggs to the same package so that this stuff could be pkg-private - line 304: // TODO: specialize based on compiled.type: for ALL and prefixes (sinkState >= 0 ) we can avoid i/o and just set bits. x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanBigArrayBlock.java (2 lines): - line 27: private static final long BASE_RAM_BYTES_USED = 0; // TODO: fix this - line 153: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/sql/sql-proto/src/main/java/org/elasticsearch/xpack/sql/proto/SqlQueryResponse.java (2 lines): - line 19: // TODO: Simplify cursor handling - line 22: // TODO investigate reusing Page here - it probably is much more efficient server/src/main/java/org/elasticsearch/plugins/PluginsLoader.java (2 lines): - line 157: // TODO: remove this leniency, but tests bogusly rely on it - line 359: Set.of("org.elasticsearch.server"), // TODO: instead of denying server, allow only jvm + stable API modules x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/store/RoleDescriptorStore.java (2 lines): - line 217: // TODO: why not populate negativeLookupCache here with missing roles? - line 219: // TODO: replace with a class that better represent the result, e.g. carry info for disabled role x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/querydsl/query/MultiMatchQuery.java (2 lines): - line 29: // TODO: it'd be great if these could be constants instead of Strings, needs a core change to make the fields public first - line 32: // TODO: add zero terms query support, I'm not sure the best way to parse it yet... server/src/main/java/org/elasticsearch/action/support/local/TransportLocalClusterStateAction.java (2 lines): - line 55: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 89: // Workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can modules/analysis-common/src/main/java/org/elasticsearch/analysis/common/AbstractCompoundWordTokenFilterFactory.java (2 lines): - line 31: // TODO expose this parameter? - line 41: // TODO is the default of true correct? see: https://github.com/apache/lucene/pull/14278 x-pack/plugin/ent-search/src/main/java/org/elasticsearch/xpack/application/connector/syncjob/ConnectorSyncJobIndexService.java (2 lines): - line 427: // TODO: don't return sensitive data from configuration inside connector in list endpoint - line 464: // TODO: what to do, if no total document count is specified? Calculate it via the current count and params of the request? fetch x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/SampleExec.java (2 lines): - line 108: // TODO: fix this. - line 110: // TODO: what should the error message here be? This doesn't seem right. server/src/main/java/org/elasticsearch/index/mapper/AbstractGeometryFieldMapper.java (2 lines): - line 85: // TODO: move geometry normalization to the geometry parser. - line 190: // TODO consider optimization using BlockSourceReader.lookupFromFieldNames(blContext.fieldNames(), name()) server/src/main/java/org/elasticsearch/injection/guice/internal/MoreTypes.java (2 lines): - line 151: // TODO: Is this sufficient? - line 183: // TODO: save a .clone() call server/src/main/java/org/elasticsearch/plugins/UberModuleClassLoader.java (2 lines): - line 115: // TODO: check that denied modules are not brought as transitive dependencies (or switch to allow-list?) - line 259: // TODO: do we need to check the security manager here? x-pack/plugin/sql/qa/server/src/main/java/org/elasticsearch/xpack/sql/qa/rest/RestSqlTestCase.java (2 lines): - line 297: // TODO: what exactly is this test suppossed to do. We need to check the 2074 issue above. - line 1789: // TODO: list aliases and data streams in CCS setups x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/mvdedupe/X-MultivalueDedupe.java.st (2 lines): - line 46: static final int ALWAYS_COPY_MISSING = 20; // TODO BytesRef should try adding to the hash *first* and then comparing. - line 72: // TODO very large numbers might want a hash based implementation - and for BytesRef that might not be that big x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/action/TransportGetRollupCapsAction.java (2 lines): - line 49: // TODO replace SAME when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 64: // Workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can libs/geo/src/main/java/org/elasticsearch/geometry/utils/SpatialEnvelopeVisitor.java (2 lines): - line 246: // TODO: Fix bug with rectangle crossing the dateline (see Extent.addRectangle for correct behaviour) - line 320: // TODO: Support circle, if given CRS (needed for radius to x/y coordinate transformation) server/src/main/java/org/elasticsearch/action/support/replication/ReplicationOperation.java (2 lines): - line 213: // TODO: fail shard? This will otherwise have the local / global checkpoint info lagging, or possibly have replicas - line 312: // Only report "critical" exceptions - TODO: Reach out to the master node to get the latest shard state then report. x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/planner/Mapper.java (2 lines): - line 103: // TODO: Translate With in a subplan - line 120: // TODO: pick up on nested/parent-child docs x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/Top.java (2 lines): - line 120: return (int) limitField().fold(FoldContext.small() /* TODO remove me */); - line 124: return BytesRefs.toString(orderField().fold(FoldContext.small() /* TODO remove me */)); x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/Outputs.java (2 lines): - line 38: // TODO: maybe change this API to allow for re-use of the - line 95: // TODO: maybe make valid(T output) public...? for asserts server/src/main/java/org/elasticsearch/cluster/ClusterStateTaskExecutor.java (2 lines): - line 119: // TODO remove all remaining usages of the published state and migrate all callers to the Runnable variant, then remove this - line 155: // TODO remove all remaining usages of the published state and migrate all callers to the Runnable variant, then remove this x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/FloatBigArrayBlock.java (2 lines): - line 27: private static final long BASE_RAM_BYTES_USED = 0; // TODO: fix this - line 132: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/common/Strings.java (2 lines): - line 767: * TODO: remove this method, it makes no sense to turn potentially very large chunked xcontent instances into a string - line 812: * TODO: remove this method, it makes no sense to turn potentially very large chunked xcontent instances into a string server/src/main/java/org/elasticsearch/index/shard/ShardSplittingQuery.java (2 lines): - line 82: // TODO IndexRouting should build the query somehow - line 135: // TODO have the IndexRouting build the query and pass routingRequired in x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/action/TransportRollupSearchAction.java (2 lines): - line 226: // TODO this filter agg is now redundant given we filter on job ID - line 466: // TODO - avoid the implicit incref elsewhere and then replace this whole thing with a ChannelActionListener x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvSort.java (2 lines): - line 248: ((BytesRef) order.fold(FoldContext.small() /* TODO remove me */)).utf8ToString() - line 257: Object obj = order.fold(FoldContext.small() /* TODO remove me */); x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/FirstOverTimeIntAggregator.java (2 lines): - line 36: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 96: // TODO: handle multiple values? x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/action/TransportGetRollupIndexCapsAction.java (2 lines): - line 57: // TODO replace SAME when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 77: // Workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can server/src/main/java/org/elasticsearch/cluster/coordination/ElasticsearchNodeCommand.java (2 lines): - line 91: // TODO: Try to parse other named objects (e.g. stored scripts, ingest pipelines) that are part of core es as well? - line 99: // TODO: Make Condition non-pluggable server/src/main/java/org/elasticsearch/common/settings/Settings.java (2 lines): - line 1241: // TODO: do this without a serialization round-trip - line 1405: // TODO We could use an FST internally to make things even faster and more compact x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/FirstOverTimeFloatAggregator.java (2 lines): - line 36: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 96: // TODO: handle multiple values? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/enrich/LookupFromIndexOperator.java (2 lines): - line 37: // TODO rename package - line 214: // TODO: Maybe create a sub-task as the parent task of all the lookup tasks build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/BwcSetupExtension.java (2 lines): - line 119: // TODO: Disabled for now until we can figure out why files are getting corrupted - line 138: // TODO resolve x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/store/DeprecationRoleDescriptorConsumer.java (2 lines): - line 200: // TODO implement failures handling in a follow-up - line 216: // TODO implement failures handling in a follow-up x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/action/saml/TransportSamlPrepareAuthenticationAction.java (2 lines): - line 45: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 63: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can server/src/main/java/org/elasticsearch/index/shard/ShardPath.java (2 lines): - line 225: // TODO: this is a hack!! We should instead keep track of incoming (relocated) shards since we know - line 232: // TODO - do we need something more extensible? Yet, this does the job for now... x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/Configuration.java (2 lines): - line 127: out.writeOptionalString(username); // TODO this one is always null - line 128: out.writeOptionalString(clusterName); // TODO this one is never null so maybe not optional x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/MachineLearning.java (2 lines): - line 590: // TODO Remove if compatibility with 8.x is no longer necessary - line 2053: // TODO should the inference indices be included here? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/categorization/GrokPatternCreator.java (2 lines): - line 64: // TODO: would be nice to have IPORHOST here, but HOST matches almost all words - line 72: // TODO: also unfortunately can't have USERNAME in the list as it matches too broadly x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntBigArrayBlock.java (2 lines): - line 27: private static final long BASE_RAM_BYTES_USED = 0; // TODO: fix this - line 132: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elasticsearch/ElasticsearchInternalService.java (2 lines): - line 186: // TODO complete deprecation of null model ID - line 441: // TODO remove this case once we remove the option to not pass model ID server/src/main/java/org/elasticsearch/cluster/service/ClusterApplierService.java (2 lines): - line 574: // TODO assert "ClusterStateApplier must not set response headers in the ClusterApplierService" - line 602: // TODO assert "ClusterStateApplier must not set response headers in the ClusterStateListener" server/src/main/java/org/elasticsearch/common/settings/KeyStoreWrapper.java (2 lines): - line 216: // TODO: this doesn't need to be a supplier anymore - line 559: // TODO: make settings accessible only to code that registered the setting x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/ApiKeyService.java (2 lines): - line 2388: // TODO we should use the effective subject realm here but need to handle the failed lookup scenario, in which the realm may be - line 2437: // TODO we should use the effective subject realm here but need to handle the failed lookup scenario, in which the realm may be server/src/main/java/org/elasticsearch/index/query/ScriptQueryBuilder.java (2 lines): - line 201: // TODO: how can we compute this? - line 211: // TODO: Change this to true when we can assume that scripts are pure functions server/src/main/java/org/elasticsearch/action/search/SearchResponse.java (2 lines): - line 97: // TODO: use DelayableWriteable instead. - line 549: // TODO: change assert to total == 1 or total = 0 - this should probably only be used for local searches now x-pack/plugin/migrate/src/main/java/org/elasticsearch/system_indices/task/SystemIndexMigrator.java (2 lines): - line 569: .prepareAliases(TimeValue.THIRTY_SECONDS, TimeValue.THIRTY_SECONDS); // TODO should these be longer? - line 888: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/AuthorizationService.java (2 lines): - line 224: // TODO the AuthorizationInfo is associated to the Subject; the argument is redundant and a possible source of conflict - line 411: // TODO: audit? x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/cohere/action/CohereActionCreator.java (2 lines): - line 36: // TODO Batching - accept a class that can handle batching - line 45: // TODO - Batching pass the batching class on to the CohereEmbeddingsRequestManager server/src/main/java/org/elasticsearch/search/lookup/FieldValues.java (2 lines): - line 75: return StoredFieldsSpec.NEEDS_SOURCE; // TODO can we get more information from the script - line 113: return StoredFieldsSpec.NEEDS_SOURCE; // TODO can we get more info from the script? server/src/main/java/org/elasticsearch/shutdown/PluginShutdownService.java (2 lines): - line 68: // TODO: consider adding debugging information (a message about why not?) - line 69: // TODO: consider adding more fine-grained status rather than true/false server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/DesiredBalanceReconciler.java (2 lines): - line 249: * TODO: We could be smarter here and group the shards by index and then - line 637: // TODO consider ignored nodes here too? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/optimizer/Optimizer.java (2 lines): - line 545: // TODO: add rule for combining Pivot with underlying project - line 557: // TODO: this need rewriting when moving functions of NamedExpression x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/InvalidArgumentException.java (2 lines): - line 13: * TODO: reason for [E|S|ES]QL specializations of QlIllegalArgumentException? - line 14: * TODO: the intended use of ql.ParsingException, vs its [E|S|ES]QL equivalents, subclassed from the respective XxxClientException? x-pack/plugin/wildcard/src/main/java/org/elasticsearch/xpack/wildcard/mapper/WildcardFieldMapper.java (2 lines): - line 502: // TODO: consider expanding this to allow for character ranges as well (need additional tests and performance eval) - line 567: // TODO optimise more intelligently so we: x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/action/saml/TransportSamlInvalidateSessionAction.java (2 lines): - line 63: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 78: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/PackedValuesBlockHash.java (2 lines): - line 204: // TODO a test where targetByteSize is very small should still make a few rows. - line 250: // TODO replace this with a warning and break plugins/discovery-gce/src/main/java/org/elasticsearch/cloud/gce/GceInstancesServiceImpl.java (2 lines): - line 70: // TODO: get this fixed - line 213: // TODO: get this fixed x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportPutTrainedModelAliasAction.java (2 lines): - line 235: // TODO should we fail in this case??? - line 254: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/PropagateInlineEvals.java (2 lines): - line 42: // TODO: replace this with AttributeSet - line 67: // TODO: look into identifying refs by their NameIds instead x-pack/plugin/profiling/src/main/java/org/elasticsearch/xpack/profiling/persistence/ProfilingDataStreamManager.java (2 lines): - line 145: TimeValue.ONE_MINUTE /* TODO should we wait longer? */, - line 146: TimeValue.THIRTY_SECONDS /* TODO should we wait longer? */, x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/store/NativeRolesStore.java (2 lines): - line 190: // TODO remove this short circuiting and fix tests that fail without this! - line 808: // TODO remove this short circuiting and fix tests that fail without this! x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/table/BlockHashRowInTableLookup.java (2 lines): - line 52: // TODO double check these errors over REST once we have LOOKUP - line 68: // TODO double check these errors over REST once we have LOOKUP x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/Enrich.java (2 lines): - line 154: out.writeString(BytesRefs.toString(policyName().fold(FoldContext.small() /* TODO remove me */))); // old policy name - line 288: * TODO: x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/topn/ResultBuilderForDoc.java (2 lines): - line 25: // TODO use fixed length builders - line 71: // TODO memory accounting x-pack/plugin/esql/qa/server/src/main/java/org/elasticsearch/xpack/esql/qa/rest/RestEsqlTestCase.java (2 lines): - line 415: // TODO: https://github.com/elastic/elasticsearch/issues/102935 - line 427: // TODO: once we do not support infinity and NaN anymore, remove INFINITY/NAN cases. server/src/main/java/org/elasticsearch/search/aggregations/bucket/filter/FiltersAggregator.java (2 lines): - line 315: // TODO: https://github.com/elastic/elasticsearch/issues/126955 - line 329: // TODO: https://github.com/elastic/elasticsearch/issues/126955 modules/data-streams/src/main/java/org/elasticsearch/datastreams/lifecycle/DataStreamLifecycleService.java (2 lines): - line 604: TimeValue.THIRTY_SECONDS /* TODO should this be longer/configurable? */, - line 672: // TODO add support for cancelling a current in-progress operation if another, later, round matches x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/common/DocumentConversionUtils.java (2 lines): - line 52: * TODO: Find out how to properly handle user-provided fields whose names start with underscore character ('_'). - line 75: // TODO: overwrites types, requires resolve if types are mixed x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/execution/ExecutionService.java (2 lines): - line 365: // TODO log watch record in logger, when saving in history store failed, otherwise the info is gone! - line 403: // TODO should we really just drop this exception on the floor? x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/FSTCompiler.java (2 lines): - line 30: // TODO: could we somehow stream an FST to disk while we - line 652: // TODO: instead of recording isFinal/output on the x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/pivot/AggregationResultUtils.java (2 lines): - line 169: // TODO: can the Percentiles extractor be removed? - line 173: // TODO: can the Range extractor be removed? libs/cli/src/main/java/org/elasticsearch/cli/Terminal.java (2 lines): - line 322: // TODO: Remove reflection once Java 22 sources are supported, e.g. using a MRJAR - line 348: // TODO: InputStreamReader can advance stdin past what it decodes. We need a way to buffer this and put it back x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/jwt/JwtSignatureValidator.java (2 lines): - line 247: // TODO: assert algorithm? - line 270: // TODO: assert algorithm? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/stats/SearchContextStats.java (2 lines): - line 184: // TODO: how to compare with the previous min - line 206: // TODO: how to compare with the previous max x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/lookup/QueryList.java (2 lines): - line 291: // TODO: support multiple values - line 323: // TODO: Support cartesian ShapeQueryable x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/LongBucketedSort.java (2 lines): - line 155: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 199: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/VariableWidthHistogramAggregator.java (2 lines): - line 282: // TODO: (maybe) Create a new bucket for all distant docs and merge down to shardSize buckets at end - line 336: * TODO: Make this more efficient server/src/main/java/org/elasticsearch/lucene/util/automaton/MinimizationOperations.java (2 lines): - line 96: // TODO moved the following into the loop because we cannot reset transition.transitionUpto (pkg private) - line 102: // TODO we'd need to be able to access pkg private transition.transitionUpto if we want to optimize the following x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/topmetrics/TopMetricsAggregator.java (2 lines): - line 315: // TODO allow configuration of value mode - line 381: // TODO allow configuration of value mode x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/InvalidArgumentException.java (2 lines): - line 13: * TODO: reason for [E|S|ES]QL specializations of QlIllegalArgumentException? - line 14: * TODO: the intended use of ql.ParsingException, vs its [E|S|ES]QL equivalents, subclassed from the respective XxxClientException? libs/x-content/src/main/java/org/elasticsearch/xcontent/ParsedMediaType.java (2 lines): - line 56: * TODO Do not rely on nulls - line 148: // TODO undefined parameters are allowed until https://github.com/elastic/elasticsearch/issues/63080 x-pack/plugin/sql/jdbc/src/main/java/org/elasticsearch/xpack/sql/jdbc/JdbcDatabaseMetaData.java (2 lines): - line 149: // TODO: is the javadoc accurate - line 180: // TODO: sync this with the grammar x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/querydsl/query/MatchQuery.java (2 lines): - line 30: // TODO: it'd be great if these could be constants instead of Strings, needs a core change to make the fields public first - line 31: // TODO: add zero terms query support, I'm not sure the best way to parse it yet... server/src/main/java/org/elasticsearch/index/mapper/DateFieldMapper.java (2 lines): - line 683: // TODO can we emit a warning if we're losing precision here? I'm not sure we can. - line 1061: // TODO maybe aggs should force millis because lots so of other places want nanos? server/src/main/java/org/elasticsearch/action/bulk/BulkRequestParser.java (2 lines): - line 72: // TODO: Remove this parameter once the BulkMonitoring endpoint has been removed - line 505: // TODO: support dynamic_templates in update requests server/src/main/java/org/elasticsearch/rest/RestRequest.java (2 lines): - line 180: // TODO: make all usages of headers case-insensitive - line 645: // TODO stop ignoring parameters such as charset... x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/action/StartTrainedModelDeploymentAction.java (2 lines): - line 61: * TODO Check if it is substantially different in other platforms. - line 448: // TODO add support for other roles? If so, it may have to be an instance method... x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportInternalInferModelAction.java (2 lines): - line 251: // TODO should `getModelForInternalInference` be used here?? - line 442: // TODO is this the correct behaviour for batched requests? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/blockhash/DoubleBlockHash.java (2 lines): - line 54: // TODO track raw counts and which implementation we pick for the profiler - #114008 - line 102: // TODO honor targetBlockSize and chunk the pages if requested. x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/blockhash/LongBlockHash.java (2 lines): - line 54: // TODO track raw counts and which implementation we pick for the profiler - #114008 - line 102: // TODO honor targetBlockSize and chunk the pages if requested. x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/transport/actions/TransportWatcherServiceAction.java (2 lines): - line 67: // TODO: make WatcherServiceRequest a real AcknowledgedRequest so that we have both a configurable timeout and master node timeout - line 103: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/execution/search/SearchHitRowSet.java (2 lines): - line 111: // TODO: add support for multi-nested doc - line 187: // TODO: improve this for multi-nested responses x-pack/plugin/esql/qa/server/single-node/src/javaRestTest/java/org/elasticsearch/xpack/esql/qa/single_node/RestEsqlIT.java (2 lines): - line 534: // TODO adapt this to use description once this is re-enabled - line 658: // TODO: Figure out how best to make sure we don't leave out new types server/src/main/java/org/elasticsearch/search/aggregations/support/CoreValuesSourceType.java (2 lines): - line 97: /* TODO: this silently ignores a timezone argument, whereas NumberFieldType#docValueFormat throws if given a time zone. - line 181: // TODO: also support the structured formats of geo points x-pack/plugin/sql/jdbc/src/main/java/org/elasticsearch/xpack/sql/jdbc/TypeConverter.java (2 lines): - line 435: // TODO: should we support conversion to TIMESTAMP? - line 599: // TODO: should we implement numeric - interval types conversions too; ever needed? ODBC does mandate it x-pack/plugin/ent-search/src/main/java/org/elasticsearch/xpack/application/analytics/event/AnalyticsEvent.java (2 lines): - line 199: // TODO: remove. Only used in tests. - line 203: // TODO: Move the interface to the package (renamed into AnalyticsContext) x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/operator/mvdedupe/MultivalueDedupeBytesRef.java (2 lines): - line 32: static final int ALWAYS_COPY_MISSING = 20; // TODO BytesRef should try adding to the hash *first* and then comparing. - line 51: // TODO very large numbers might want a hash based implementation - and for BytesRef that might not be that big x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/grouping/Bucket.java (2 lines): - line 67: // TODO maybe we should just cover the whole of representable dates here - like ten years, 100 years, 1000 years, all the way up. - line 90: private static final ZoneId DEFAULT_TZ = ZoneOffset.UTC; // TODO: plug in the config server/src/main/java/org/elasticsearch/cluster/action/shard/ShardStateAction.java (2 lines): - line 281: // TODO: Make this a TransportMasterNodeAction and remove duplication of master failover retrying from upstream code - line 592: // TODO: Make this a TransportMasterNodeAction and remove duplication of master failover retrying from upstream code x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/DoubleBucketedSort.java (2 lines): - line 155: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 199: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting server/src/main/java/org/elasticsearch/index/codec/vectors/ES814ScalarQuantizedVectorsFormat.java (2 lines): - line 259: // TODO: optimize int4 quantization - line 282: // TODO: optimize int4 quantization server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/TermsAggregatorFactory.java (2 lines): - line 130: // TODO this exception message is not really accurate for the string case. It's really disallowing regex + formatter - line 140: // TODO: [Zach] we might want refactor and remove ExecutionMode#create(), moving that logic outside the enum x-pack/plugin/ccr/src/javaRestTest/java/org/elasticsearch/xpack/ccr/AutoFollowIT.java (2 lines): - line 734: // TODO: verify that following a backing index for logs-tomcat-prod data stream in remote cluster fails, - line 940: // TODO: Don't update logs-http alias in follower cluster when data streams are automatically replicated x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/frequentitemsets/TransactionStore.java (2 lines): - line 301: // TODO: heap based and wasteful, this should instead use some lucene magic if possible - line 339: // TODO: heap based and wasteful, this should instead use some lucene magic server/src/main/java/org/elasticsearch/persistent/AllocatedPersistentTask.java (2 lines): - line 72: TimeValue.THIRTY_SECONDS /* TODO should this be longer? infinite? */, - line 210: TimeValue.THIRTY_SECONDS /* TODO should this be longer? infinite? */, x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongBigArrayBlock.java (2 lines): - line 27: private static final long BASE_RAM_BYTES_USED = 0; // TODO: fix this - line 132: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/internalClusterTest/java/org/elasticsearch/indices/IndicesOptionsIntegrationIT.java (2 lines): - line 272: // TODO: temporary work-around for #5531 - line 386: // TODO: temporary work-around for #5531 x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/X-ValueOverTimeAggregator.java.st (2 lines): - line 40: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 100: // TODO: handle multiple values? server/src/main/java/org/elasticsearch/cluster/routing/RoutingNodes.java (2 lines): - line 221: // TODO: check primary == null || primary.active() after all tests properly add ReplicaAfterPrimaryActiveAllocationDecider - line 681: false, // TODO debatable, but do we want to delay reassignment of unpromotable replicas tho? server/src/main/java/org/elasticsearch/search/aggregations/support/ValuesSourceAggregationBuilder.java (2 lines): - line 242: // TODO: deprecate this so we don't need to carry around a useless null in the wire format - line 326: // TODO: This is nonsense. We allow the value to be null (via constructor), but don't allow it to be set to null. This means x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/blockhash/IntBlockHash.java (2 lines): - line 52: // TODO track raw counts and which implementation we pick for the profiler - #114008 - line 100: // TODO honor targetBlockSize and chunk the pages if requested. server/src/main/java/org/elasticsearch/index/mapper/DynamicTemplate.java (2 lines): - line 500: // TODO: remove this in 6.0 - line 501: // TODO: how to do it in the future? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/physical/local/PushTopNToSource.java (2 lines): - line 143: // TODO: can we support CARTESIAN also? - line 202: // TODO: https://github.com/elastic/elasticsearch/issues/120219 x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/job/config/Job.java (2 lines): - line 199: // TODO: Use java.time for the Dates here: x-pack-elasticsearch#829 - line 640: // TODO in v8.0.0 move this out so that it will be included when `exclude_generated` is `true` x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/CrossClusterQueryIT.java (2 lines): - line 162: // MP TODO: am I able to fix this from the field-caps call? Yes, if we detect concrete vs. wildcard expressions in user query - line 163: // TODO bug - this does not throw; uncomment this test once https://github.com/elastic/elasticsearch/issues/114495 is fixed x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/sort/FloatBucketedSort.java (2 lines): - line 155: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 199: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/X-BigArrayBlock.java.st (2 lines): - line 27: private static final long BASE_RAM_BYTES_USED = 0; // TODO: fix this - line 158: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/frequentitemsets/HashBasedTransactionStore.java (2 lines): - line 68: * - TODO: pruning is only used after merging all stores from all shards, it could be beneficial to prune already on the shard level - line 323: * TODO: server/src/main/java/org/elasticsearch/action/support/replication/TransportReplicationAction.java (2 lines): - line 196: // TODO: consider passing the executor, investigate doExecute and let InboundHandler/TransportAction handle concurrency. - line 709: // TODO: Evaluate if we still need to catch this exception server/src/main/java/org/elasticsearch/injection/guice/InjectorImpl.java (2 lines): - line 284: // TODO: for the above example, remove the binding for BarImpl if the binding for FooImpl fails - line 320: // TODO: Method interceptors could actually enable us to implement server/src/main/java/org/elasticsearch/bootstrap/Elasticsearch.java (2 lines): - line 112: // TODO: remove this, just for debugging - line 450: * TODO: should we fail hard here if system call filters fail to install, or remain lenient in non-production environments? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/sort/X-BucketedSort.java.st (2 lines): - line 155: // TODO: This can be improved for heapified buckets by making use of the heap structures - line 199: // TODO: Make use of heap structures to faster iterate in order instead of copying and sorting server/src/main/java/org/elasticsearch/index/codec/vectors/es818/ES818BinaryQuantizedVectorsWriter.java (2 lines): - line 616: // TODO Panama sum - line 625: // TODO Panama div x-pack/plugin/sql/sql-action/src/main/java/org/elasticsearch/xpack/sql/action/SqlQueryResponse.java (2 lines): - line 42: // TODO: Simplify cursor handling - line 48: // TODO investigate reusing Page here - it probably is much more efficient x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/ComputeService.java (2 lines): - line 363: execInfo.markEndQuery(); // TODO: revisit this time recording model as part of INLINESTATS improvements - line 506: execInfo.markEndQuery(); // TODO: revisit this time recording model as part of INLINESTATS improvements server/src/main/java/org/elasticsearch/index/engine/LuceneSyntheticSourceChangesSnapshot.java (2 lines): - line 196: // TODO: instead of building an array, consider just checking whether doc ids are dense. - line 243: // TODO: Callers should ask for the range that source should be retained. Thus we should always x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/FirstOverTimeDoubleAggregator.java (2 lines): - line 36: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 96: // TODO: handle multiple values? server/src/main/java/org/elasticsearch/cluster/metadata/MetadataIndexTemplateService.java (2 lines): - line 757: // TODO come up with a plan how to validate usage of deprecated ILM policies - line 1192: // TODO: in 8.0+, only allow updating index templates, not adding new ones x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/IndicesAndAliasesResolver.java (2 lines): - line 206: // TODO: Shard level requests should already have their selectors resolved to concrete indices by their parent requests - line 215: // TODO: Shard level requests have wildcard expanded already and do not need go through this check x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/action/saml/TransportSamlAuthenticateAction.java (2 lines): - line 55: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 72: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/querydsl/query/NestedQuery.java (2 lines): - line 34: // TODO: make this configurable - line 96: // TODO: Add all filters in nested sorting when https://github.com/elastic/elasticsearch/issues/33079 is implemented x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/FirstOverTimeLongAggregator.java (2 lines): - line 34: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 94: // TODO: handle multiple values? x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/InitialNodeSecurityAutoConfiguration.java (2 lines): - line 68: // TODO maybe we can improve the "did pre-start-up config run" check - line 96: // TODO maybe we can improve the check that this is indeed the initial node x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/audit/logfile/LoggingAuditTrail.java (2 lines): - line 1689: // TODO: API key can run-as which means we could use extra fields (#84394) - line 1697: // TODO: service token info is logged in a separate authentication field (#84394) modules/ingest-geoip/src/main/java/org/elasticsearch/ingest/geoip/DatabaseNodeService.java (2 lines): - line 349: // TODO perhaps we need to handle the license flap persistent task state better than we do - line 509: // TODO: invoke open point in time api when this api is moved from xpack core to server module. server/src/main/java/org/elasticsearch/action/support/broadcast/node/TransportBroadcastByNodeAction.java (2 lines): - line 111: // TODO replace SAME when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 - line 234: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can x-pack/plugin/text-structure/src/main/java/org/elasticsearch/xpack/textstructure/structurefinder/DelimitedTextStructureFinder.java (2 lines): - line 682: // TODO: might be good one day to gather a distribution of the most common field counts - line 848: // TODO: if this happens a lot then we should try looking for the a multi-line END pattern instead of a start pattern. x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/async/AsyncTaskManagementService.java (2 lines): - line 281: // TODO: generalize the logging, not just eql - line 300: // TODO: generalize the logging, not just eql server/src/main/java/org/elasticsearch/action/admin/cluster/repositories/cleanup/TransportCleanupRepositoryAction.java (2 lines): - line 58: * and any delete/write operations. TODO: This will not work if we also want to clean up at the shard level as those will involve writes - line 296: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/common/time/EpochTime.java (2 lines): - line 45: // FIXME: what should this be? - line 283: // FIXME: clean these up and append one to the other x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/type/DataType.java (2 lines): - line 385: // TODO: Why don't we use the names ES uses as the esType field for these? - line 667: // TODO: Use our normal enum serialization pattern server/src/main/java/org/elasticsearch/action/admin/cluster/snapshots/get/TransportGetSnapshotsAction.java (2 lines): - line 300: // TODO if the request parameters allow it, modify asyncSnapshotInfoIterators to skip unnecessary GET calls here - line 711: // TODO: this could be made more efficient by caching this number in RepositoryData x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/DataNodeComputeHandler.java (2 lines): - line 284: // TODO: add these to fatal failures so we can continue processing other shards. - line 344: // TODO: `searchService.createSearchContext` allows opening search contexts without limits, x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/lookup/RightChunkedLeftJoin.java (2 lines): - line 218: // TODO make a filter that takes a min and max? - line 254: // TODO replace parameter to filter with vector and remove this x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/store/ReservedRolesStore.java (2 lines): - line 81: // TODO add read_failure_store when failures authorization is implemented - line 99: // TODO add read_failure_store when failures authorization is implemented x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/plugin/TransportEqlSearchAction.java (2 lines): - line 184: // TODO: these should be sent by the client - line 301: // remote or not, nor which remote cluster it belongs to (TODO: could rewrite the ID to smth like [alias:ID]) x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/ColumnLoadOperator.java (2 lines): - line 53: * TODO target the size more intelligently - line 59: // TODO tracking is complex for values server/src/main/java/org/elasticsearch/index/codec/vectors/SampleReader.java (2 lines): - line 83: // TODO can we do something algorithmically that aligns an ordinal with a unique integer between 0 and numVectors? - line 87: // TODO maybe use bigArrays? server/src/main/java/org/elasticsearch/cluster/metadata/MetadataCreateIndexService.java (2 lines): - line 367: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here - line 634: // TODO: this method can be removed in 9.0 because we will no longer use v1 templates to create indices (only v2 templates) x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/LastOverTimeIntAggregator.java (2 lines): - line 36: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 96: // TODO: handle multiple values? plugins/analysis-smartcn/src/main/java/org/elasticsearch/plugin/analysis/smartcn/AnalysisSmartChinesePlugin.java (2 lines): - line 30: // TODO: deprecate and remove, this is a noop token filter; it's here for backwards compat before we had "smartcn_tokenizer" - line 39: // TODO: deprecate and remove, this is an alias to "smartcn_tokenizer"; it's here for backwards compat x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/LastOverTimeLongAggregator.java (2 lines): - line 34: // TODO: Since data in data_streams is sorted by `_tsid` and timestamp in descending order, - line 94: // TODO: handle multiple values? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/NerProcessor.java (2 lines): - line 187: // TODO - process all results in the batch - line 192: // TODO It might be best to do the soft max after averaging scores for server/src/main/java/org/elasticsearch/search/sort/ScriptSortBuilder.java (1 line): - line 243: // TODO there must be a better way to get the field type... server/src/main/java/org/elasticsearch/rest/action/RestToXContentListener.java (1 line): - line 24: // TODO make this final x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/AsyncOperator.java (1 line): - line 222: // TODO: Add an exchange service between async operation instead? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/plan/logical/Pivot.java (1 line): - line 130: // FIXME: the value attributes are reused and thus will clash - new ids need to be created x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/Median.java (1 line): - line 36: // TODO: Add the compression parameter x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/highlight/SemanticTextHighlighter.java (1 line): - line 251: // TODO: Handle knn section when semantic text field can be used. server/src/main/java/org/elasticsearch/injection/guice/internal/ConstructionContext.java (1 line): - line 64: // TODO: if I create a proxy which implements all the interfaces of server/src/internalClusterTest/java/org/elasticsearch/cluster/routing/allocation/decider/DiskThresholdDeciderIT.java (1 line): - line 353: // even though it's now possible to achieve better balance, so we have to do an explicit reroute. TODO fix this? x-pack/plugin/migrate/src/main/java/org/elasticsearch/system_indices/task/MigrationResultsUpdateTask.java (1 line): - line 62: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/Sample.java (1 line): - line 167: return (int) limitField().fold(FoldContext.small() /* TODO remove me */); x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/autoscaling/MlAutoscalingContext.java (1 line): - line 94: // TODO: Eventually care about those that are STARTED but not FULLY_ALLOCATED server/src/main/java/org/elasticsearch/node/Node.java (1 line): - line 326: // TODO: Do not expect that the legacy metadata file is always present https://github.com/elastic/elasticsearch/issues/95211 x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authc/Subject.java (1 line): - line 238: // TODO: should we validate enable status and length of role names on instantiation time of anonymousUser? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/tokenizers/WordPieceTokenFilter.java (1 line): - line 94: // TODO seems like this + lowercase + tokenize cjk + wordpiece could all be the same thing.... server/src/main/java/org/elasticsearch/TransportVersions.java (1 line): - line 51: // TODO: ES-10337 we can remove all transport versions earlier than 8.18 x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/TransportUnfollowAction.java (1 line): - line 241: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/core/src/main/java/org/elasticsearch/license/RestPutLicenseAction.java (1 line): - line 29: // TODO: remove POST endpoint? server/src/main/java/org/elasticsearch/cluster/routing/allocation/decider/ThrottlingAllocationDecider.java (1 line): - line 136: // TODO: Should index creation not be throttled for primary shards? x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/EsqlAsyncActionIT.java (1 line): - line 127: // -- TODO: eventually remove and use common compute test infra server/src/main/java/org/elasticsearch/index/codec/bloomfilter/ES87BloomFilterPostingsFormat.java (1 line): - line 464: // TODO: return TermState that includes BloomFilter and fix _disk_usage API server/src/main/java/org/elasticsearch/env/Environment.java (1 line): - line 35: // TODO: move PathUtils to be package-private here instead of x-pack/plugin/autoscaling/src/main/java/org/elasticsearch/xpack/autoscaling/action/TransportPutAutoscalingPolicyAction.java (1 line): - line 102: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/text-structure/src/main/java/org/elasticsearch/xpack/textstructure/transport/TextStructExecutor.java (1 line): - line 22: * TODO delete this entire class when we can x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/oidc/OpenIdConnectAuthenticator.java (1 line): - line 488: // TODO Handle validating possibly signed responses x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/rest/action/user/RestSetEnabledAction.java (1 line): - line 56: // TODO consider splitting up enable and disable to have their own rest handler server/src/main/java/org/elasticsearch/search/internal/FieldUsageTrackingDirectoryReader.java (1 line): - line 152: // TODO: fix in Lucene server/src/main/java/org/elasticsearch/search/aggregations/support/AggregationContext.java (1 line): - line 249: // TODO it is unclear why we can't just use the IndexSearcher which already caches x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/searchablesnapshots/MountSearchableSnapshotRequest.java (1 line): - line 64: * TODO: remove in 9.0. x-pack/plugin/enrich/src/internalClusterTest/java/org/elasticsearch/xpack/enrich/EnrichRestartIT.java (1 line): - line 43: // TODO Change this to run with security enabled x-pack/plugin/autoscaling/src/main/java/org/elasticsearch/xpack/autoscaling/action/TransportDeleteAutoscalingPolicyAction.java (1 line): - line 78: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/physical/local/LucenePushdownPredicates.java (1 line): - line 118: // TODO: This is the original behaviour, but is it correct? In FieldType isAggregatable usually only means hasDocValues x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/NodeHash.java (1 line): - line 91: // TODO: maybe if number of arcs is high we can safely subsample? server/src/main/java/org/elasticsearch/search/fetch/FetchPhase.java (1 line): - line 307: // TODO: can we use real pooled buffers here as well? server/src/main/java/org/elasticsearch/action/search/BottomSortValuesCollector.java (1 line): - line 80: // TODO: we should fail the entire request because the topdocs x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/IntFallibleArrayState.java (1 line): - line 109: valuesBuilder.appendInt(0); // TODO can we just use null? x-pack/plugin/kql/src/main/java/org/elasticsearch/xpack/kql/parser/KqlAstBuilder.java (1 line): - line 67: // TODO: KQLContext has an option to wrap the clauses into a filter instead of a must clause. Do we need it? x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/xpack/searchablesnapshots/store/input/MetadataCachingIndexInput.java (1 line): - line 305: // TODO TBD use a different trigger for creating the cache index and avoid a put in the CACHE_NOT_READY case. server/src/main/java/org/elasticsearch/search/rank/feature/RankFeatureShardPhase.java (1 line): - line 74: // TODO: here we populate the profile part of the fetchResult as well server/src/main/java/org/elasticsearch/health/RestGetHealthAction.java (1 line): - line 38: // TODO: Existing - "cluster_health_action", "cat_health_action" x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/trainedmodel/tree/Tree.java (1 line): - line 40: // TODO should we have regression/classification sub-classes that accept the builder? x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/aggregations/support/AnalyticsValuesSourceType.java (1 line): - line 26: // TODO: Is this the correct exception type here? server/src/main/java/org/elasticsearch/index/mapper/KeywordFieldMapper.java (1 line): - line 780: // TODO what if the value didn't change? x-pack/plugin/ccr/src/javaRestTest/java/org/elasticsearch/xpack/ccr/FollowIndexSecurityIT.java (1 line): - line 333: // TODO: We're implicitly relying on index templates from the stack module here. This requires us to install this module x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/BlockHash.java (1 line): - line 112: // TODO merge with nonEmpty x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene40/blocktree/FieldReader.java (1 line): - line 186: // TODO: we could push "it's a range" or "it's a prefix" down into IntersectTermsEnum? server/src/main/java/org/elasticsearch/index/reindex/LeaderBulkByScrollTaskState.java (1 line): - line 105: // TODO cancel when a slice fails? server/src/main/java/org/elasticsearch/transport/TransportLogger.java (1 line): - line 136: // TODO: Maybe Fix for BWC server/src/main/java/org/elasticsearch/common/settings/LocallyMountedSecrets.java (1 line): - line 168: // TODO: Add support for watching for file changes here. x-pack/plugin/sql/jdbc/src/main/java/org/elasticsearch/xpack/sql/jdbc/SqlQueryParameterAnalyzer.java (1 line): - line 66: // TODO: JDBC escape syntax x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/util/SpatialCoordinateTypes.java (1 line): - line 101: // TODO: we should be able to transform WKT to WKB without building the geometry libs/geo/src/main/java/org/elasticsearch/geometry/utils/WellKnownText.java (1 line): - line 649: // TODO: Add 3D support modules/legacy-geo/src/main/java/org/elasticsearch/legacygeo/mapper/LegacyGeoShapeFieldMapper.java (1 line): - line 87: * TODO: Remove this class once we no longer need to supported reading 7.x indices that might have this field type x-pack/plugin/ent-search/src/main/java/org/elasticsearch/xpack/application/rules/action/TestQueryRulesetAction.java (1 line): - line 37: // TODO - We'd like to transition this to require less stringent permissions x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/MachineLearningExtensionHolder.java (1 line): - line 17: * TODO: remove this class entirely once Guice is removed entirely. x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/job/IndexerUtils.java (1 line): - line 63: // TODO would be nice to avoid allocating this treemap in the future x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/inference/results/SparseEmbeddingResults.java (1 line): - line 179: // TODO: maybe truncate tokens here when it's getting too large? server/src/main/java/org/elasticsearch/action/bulk/TransportAbstractBulkAction.java (1 line): - line 250: // TODO perhaps this should use `threadPool.absoluteTimeInMillis()`, but leaving as is for now. server/src/main/java/org/elasticsearch/persistent/ClusterPersistentTasksCustomMetadata.java (1 line): - line 64: // TODO: Implement custom Diff for tasks server/src/main/java/org/elasticsearch/search/aggregations/bucket/BucketsAggregator.java (1 line): - line 339: // TODO: We should instrument this error. While it is correct for it to be a 400 class IllegalArgumentException, there is server/src/main/java/org/elasticsearch/search/internal/SearchContext.java (1 line): - line 401: public void addReleasable(Releasable releasable) { // TODO most Releasables are managed by their callers. We probably don't need this. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/trainedmodel/Tokenization.java (1 line): - line 73: // TODO add global params like never_split, bos_token, eos_token, mask_token, tokenize_chinese_chars, strip_accents, etc. server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/RangeIT.java (1 line): - line 876: // TODO: use diamond once JI-9019884 is fixed x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/session/SqlSession.java (1 line): - line 148: // TODO we plan to support joins in the future when possible, but for now we'll just fail early if we see one server/src/main/java/org/elasticsearch/action/search/ClearScrollController.java (1 line): - line 58: // TODO: replace this with #closeContexts x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/cohere/CohereService.java (1 line): - line 82: // TODO Batching - We'll instantiate a batching class within the services that want to support it and pass it through to server/src/main/java/org/elasticsearch/common/util/concurrent/ListenableFuture.java (1 line): - line 27: // The name {@link ListenableFuture} dates back a long way and could be improved - TODO find a better name x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/transform/script/WatcherTransformScript.java (1 line): - line 25: // TODO: ctx should have its members extracted into execute parameters, but it needs to be a member bwc access in params server/src/main/java/org/elasticsearch/cluster/metadata/MetadataIndexAliasesService.java (1 line): - line 95: taskQueue.submitTask("index-aliases", new ApplyAliasesTask(request, listener), null); // TODO use request.masterNodeTimeout() here? libs/x-content/src/main/java/org/elasticsearch/xcontent/Text.java (1 line): - line 132: // TODO: TextBytesOptimization we can use a buffer here to convert it? maybe add a x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/fulltext/Term.java (1 line): - line 113: if (t.field() instanceof FieldAttribute == false) { // TODO: is a conversion possible, similar to Match’s case? x-pack/plugin/spatial/src/internalClusterTest/java/org/elasticsearch/xpack/spatial/search/TestSpatialQueryBuilders.java (1 line): - line 19: // TODO: Implement QueryBuilder for distance, boundingBox, polygon x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/TranslatorHandler.java (1 line): - line 57: // TODO: is this method necessary? x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/function/scalar/IntervalScripting.java (1 line): - line 10: // FIXME: accessor interface until making script generation pluggable x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportMoveToStepAction.java (1 line): - line 178: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/SearchSlowLog.java (1 line): - line 115: * TODO: Remove in 9.0 x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/CrossClusterCancellationIT.java (1 line): - line 170: // TODO: This produces incorrect results because data on the remote cluster is processed multiple times. x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportPutTrainedModelVocabularyAction.java (1 line): - line 94: // TODO do we really need to do this??? server/src/main/java/org/elasticsearch/action/admin/cluster/stats/ClusterStatsNodes.java (1 line): - line 188: // TODO: do we need to report zeros? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/tokenizers/ControlCharFilter.java (1 line): - line 24: // TODO this is probably not ultimately necessary, keeping track of where we are in the stream x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/service/IndexServiceAccountTokenStore.java (1 line): - line 190: // TODO: wildcard support? server/src/main/java/org/elasticsearch/search/aggregations/metrics/GlobalOrdCardinalityAggregator.java (1 line): - line 168: // TODO: iterate the bitset using a `nextClearBit` operation? x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/analysis/PreAnalyzer.java (1 line): - line 30: // FIXME: includeFrozen needs to be set already modules/lang-painless/src/main/java/org/elasticsearch/painless/action/PainlessExecuteAction.java (1 line): - line 525: // TODO: is MANAGEMENT TP the right TP? Right now this is an admin api (see action name). x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/FloatArrayBlock.java (1 line): - line 134: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/RollupJobIdentifierUtils.java (1 line): - line 66: // TODO there is an opportunity to optimize the returned caps to find the minimal set of required caps. server/src/internalClusterTest/java/org/elasticsearch/nodescapabilities/SimpleNodesCapabilitiesIT.java (1 line): - line 53: // TODO this is not working yet - see https://github.com/elastic/elasticsearch/issues/107425 x-pack/plugin/sql/sql-client/src/main/java/org/elasticsearch/xpack/sql/client/HttpClient.java (1 line): - line 93: // TODO allow customizing the time zone - this is what session set/reset/get should be about x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/index/mapper/ShapeFieldMapper.java (1 line): - line 255: // TODO: Make common with the index method GeoShapeWithDocValuesFieldMapper modules/repository-azure/src/internalClusterTest/java/org/elasticsearch/repositories/azure/AzureBlobStoreRepositoryTests.java (1 line): - line 201: null /* no auth header validation - sometimes it's omitted in these tests (TODO why?) */, x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/X-ConstantVector.java.st (1 line): - line 97: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/SecuritySettings.java (1 line): - line 45: Optional userOptional = SecurityField.USER_SETTING.get(settings); // TODO migrate to securesetting! x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/retention/ExpiredModelSnapshotsRemover.java (1 line): - line 163: // TODO: delete this test if we ever allow users to revert a job to no model snapshot, e.g. to recover from data loss x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/pytorch/process/PyTorchResultProcessor.java (1 line): - line 284: // TODO min inference time libs/native/src/main/java/org/elasticsearch/nativeaccess/MacNativeAccess.java (1 line): - line 62: // TODO: log warning? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/session/SingletonRowSet.java (1 line): - line 11: //TODO is it worth keeping this when we have ListRowSet? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvDedupe.java (1 line): - line 34: // @TODO: add unsigned_long x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/lucene/LuceneSourceOperator.java (1 line): - line 85: // TODO: use a single limiter for multiple stage execution server/src/main/java/org/elasticsearch/node/InternalSettingsPreparer.java (1 line): - line 74: // TODO: this fallback is only needed for tests, in production input is always Settings.EMPTY x-pack/plugin/snapshot-based-recoveries/qa/azure/src/javaRestTest/java/org/elasticsearch/xpack/snapshotbasedrecoveries/recovery/AzureSnapshotBasedRecoveryIT.java (1 line): - line 28: // TODO when https://github.com/elastic/elasticsearch/issues/111532 addressed, use a HTTPS fixture in FIPS mode too x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/StringExtractOperator.java (1 line): - line 46: private final Function> parser; // TODO parser should consume ByteRef instead of String x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/PropagateEvalFoldables.java (1 line): - line 52: // TODO: also allow aggregates once aggs on constants are supported. server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/GlobalOrdinalValuesSource.java (1 line): - line 503: // TODO If this is the only source, we know we are done with the buckets of the after_key. libs/entitlement/src/main/java/org/elasticsearch/entitlement/runtime/policy/PolicyUtils.java (1 line): - line 125: // TODO: should this check actually be part of the parser? x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/pivot/GroupByOptimizer.java (1 line): - line 31: * TODO: take index sorting into account x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportStopILMAction.java (1 line): - line 58: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/X-ArrayState.java.st (1 line): - line 149: valuesBuilder.append$Type$($if(boolean)$false$else$0$endif$); // TODO can we just use null? x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/aggregations/metrics/HistoBackedAvgAggregator.java (1 line): - line 52: // TODO: Stop depending on nulls here x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/action/PutTrainedModelAction.java (1 line): - line 72: // TODO: remove this constructor after re-factoring ML parts server/src/internalClusterTest/java/org/elasticsearch/search/fetch/subphase/highlight/HighlighterSearchIT.java (1 line): - line 111: // TODO as we move analyzers out of the core we need to move some of these into HighlighterWithAnalyzersTests server/src/main/java/org/elasticsearch/search/aggregations/metrics/TopHitsAggregator.java (1 line): - line 145: // TODO: can we pass trackTotalHits=subSearchContext.trackTotalHits(){ server/src/main/java/org/elasticsearch/search/suggest/term/TermSuggester.java (1 line): - line 43: // TODO: Extend DirectSpellChecker in 4.1, to get the raw suggested words as BytesRef server/src/main/java/org/elasticsearch/action/fieldcaps/FieldCapabilitiesFetcher.java (1 line): - line 194: // TODO find a way to do this that does not require an instanceof check x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/WildcardLike.java (1 line): - line 133: // TODO: see whether escaping is needed server/src/main/java/org/elasticsearch/repositories/IndexSnapshotsService.java (1 line): - line 133: EsExecutors.DIRECT_EXECUTOR_SERVICE, // TODO contemplate threading here, do we need to fork, see #101445? server/src/main/java/org/elasticsearch/cluster/service/ClusterService.java (1 line): - line 159: * TODO: Should be renamed to appliedState / appliedClusterState server/src/main/java/org/elasticsearch/env/ESFileStore.java (1 line): - line 37: // TODO: move PathUtils to be package-private here instead of x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/type/DataTypeConverter.java (1 line): - line 451: // TODO floating point conversions are lossy but conversions to integer are not. Are we ok with that? server/src/main/java/org/elasticsearch/index/similarity/SimilarityProviders.java (1 line): - line 71: // TODO: be and g and both based on the bose-einstein model. server/src/main/java/org/elasticsearch/gateway/GatewayMetaState.java (1 line): - line 557: // by this version of Elasticsearch. TODO TBD should we avoid indexing when possible? server/src/main/java/org/elasticsearch/index/mapper/vectors/DenseVectorFieldMapper.java (1 line): - line 1833: // TODO: add support for updating from flat, hnsw, and int8_hnsw and updating params x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/history/SnapshotLifecycleTemplateRegistry.java (1 line): - line 104: return INDEX_LIFECYCLE_ORIGIN; // TODO use separate SLM origin? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlSearchShardsAction.java (1 line): - line 42: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/execution/sequence/StageToKeys.java (1 line): - line 36: // TODO: could we use an allocation strategy? server/src/main/java/org/elasticsearch/common/network/NetworkModule.java (1 line): - line 188: // TODO: consider moving this to the ClusterModule server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/GeoDistanceIT.java (1 line): - line 423: // TODO: use diamond once JI-9019884 is fixed build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/precommit/SplitPackagesAuditTask.java (1 line): - line 275: // TODO: want to read packages the same for src dirs and jars, but src dirs we also want the files in the src package dir x-pack/plugin/repositories-metering-api/qa/azure/src/javaRestTest/java/org/elasticsearch/xpack/repositories/metering/azure/AzureRepositoriesMeteringIT.java (1 line): - line 28: // TODO when https://github.com/elastic/elasticsearch/issues/111532 addressed, use a HTTPS fixture in FIPS mode too server/src/main/java/org/elasticsearch/action/admin/cluster/stats/CCSTelemetrySnapshot.java (1 line): - line 319: // TODO: this may be sub-optimal, as we'll be copying histograms when adding first snapshot to an empty container, x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/CategorizeBlockHash.java (1 line): - line 207: // TODO: This BytesStreamOutput is not accounted for by the circuit breaker. Fix that! server/src/main/java/org/elasticsearch/search/fetch/FetchContext.java (1 line): - line 61: // TODO it seems a bit odd that we disable implicit source loading if we've asked x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/process/logging/CppLogMessageHandler.java (1 line): - line 320: // TODO: Is there a way to preserve the original timestamp when re-logging? server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/SignificantTextAggregatorFactory.java (1 line): - line 170: // TODO - need to check with mapping that this is indeed a text field.... x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/util/MathUtils.java (1 line): - line 17: // TODO: can this function be removed? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntBigArrayVector.java (1 line): - line 145: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/inference/results/TextEmbeddingBitResults.java (1 line): - line 44: // Embedding.merge method for bits. TODO: implement a proper merge method x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authc/jwt/JwtRealmSettings.java (1 line): - line 525: // TODO allow HTTPS once https://github.com/elastic/elasticsearch/issues/100264 is fixed modules/lang-painless/src/main/java/org/elasticsearch/painless/phase/DefaultSemanticAnalysisPhase.java (1 line): - line 3065: // TODO: remove ZonedDateTime exception when JodaCompatibleDateTime is removed libs/x-content/impl/src/main/java/org/elasticsearch/xcontent/provider/json/JsonXContentGenerator.java (1 line): - line 374: // TODO: this can probably be removed after upgrading Jackson to 2.15.1 or later, see modules/ingest-geoip/src/main/java/org/elasticsearch/ingest/geoip/GeoIpProcessor.java (1 line): - line 41: + "in a future version of Elasticsearch"; // TODO add a message about migration? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/io/stream/PlanStreamOutput.java (1 line): - line 44: * TODO should this be a cluster setting...? x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/action/TransportStartSLMAction.java (1 line): - line 62: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authc/support/UsernamePasswordToken.java (1 line): - line 43: // TODO we still have passwords in Strings in headers. Maybe we can look into using a CharSequence? server/src/main/java/org/elasticsearch/index/mapper/SourceFieldBlockLoader.java (1 line): - line 56: // TODO support appending BytesReference server/src/main/java/org/elasticsearch/script/field/vectors/ByteKnnDenseVector.java (1 line): - line 32: // TODO it would be really nice if we didn't transform the `byte[]` arrays to `float[]` x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/lucene/LuceneCountOperator.java (1 line): - line 142: // TODO: avoid iterating all documents in multiple calls to make cancellation more responsive. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/privilege/ManageOwnApiKeyClusterPrivilege.java (1 line): - line 136: * TODO bizybot we need to think on how we can propagate appropriate error message to the end user when username, realm name x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/plan/logical/command/sys/SysColumns.java (1 line): - line 250: // TODO: is the buffer_length correct? x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/ClientTransformIndexer.java (1 line): - line 136: // TODO: move into context constructor x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/rollup/job/MetricConfig.java (1 line): - line 48: // TODO: replace these with an enum server/src/main/java/org/elasticsearch/cluster/DiffableUtils.java (1 line): - line 386: // TODO: refactor map diff reading to avoid having to construct empty diffs before throwing them away here x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/ConstantLongVector.java (1 line): - line 62: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. libs/tdigest/src/main/java/org/elasticsearch/tdigest/IntAVLTree.java (1 line): - line 316: // TODO: it should be possible to find the new node position without x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/dataframe/analyses/Classification.java (1 line): - line 115: (classification) -> {/*TODO should we throw if this is not set?*/}, x-pack/plugin/searchable-snapshots/src/internalClusterTest/java/org/elasticsearch/xpack/searchablesnapshots/ClusterStateApplierOrderingTests.java (1 line): - line 47: // we can bypass this by forcing soft deletes to be used. TODO this restriction can be lifted when #55142 is resolved. x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/categorization/CategorizationPartOfSpeechDictionary.java (1 line): - line 127: // TODO: now we have this in Java, perform this operation in Java for anomaly detection categorization instead of in C++. server/src/main/java/org/elasticsearch/search/sort/SortBuilder.java (1 line): - line 60: // TODO: this can deadlock as it might access the ScoreSortBuilder (subclass) initializer from the SortBuilder initializer!!! x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/persistence/IndexBasedTransformConfigManager.java (1 line): - line 107: * TODO: Provide a method that moves old docs into the current index and delete old indexes and templates x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/FloatLookup.java (1 line): - line 86: // TODO replace this with a warning and break server/src/main/java/org/elasticsearch/action/admin/cluster/configuration/TransportClearVotingConfigExclusionsAction.java (1 line): - line 167: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/BooleanFallibleArrayState.java (1 line): - line 110: valuesBuilder.appendBoolean(false); // TODO can we just use null? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/BooleanSimplification.java (1 line): - line 129: // TODO: eliminate conjunction/disjunction x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/aggregations/bucket/range/HistoBackedRangeAggregator.java (1 line): - line 34: // TODO it would be good one day to possibly interpolate between ranges in the histogram fields x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/index/mapper/GeoShapeWithDocValuesFieldMapper.java (1 line): - line 406: // TODO: Make common with the index method ShapeFieldMapper x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/expression/function/Function.java (1 line): - line 27: // TODO: Functions supporting distinct should add a dedicated constructor Location, List, boolean server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/RareTermsAggregationBuilder.java (1 line): - line 141: // TODO review: what size cap should we put on this? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/HashJoinExec.java (1 line): - line 121: // TODO: this is a hack until qualifiers land since the right side is always materialized x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/predicate/operator/comparison/LessThan.java (1 line): - line 138: @Evaluator(extraName = "Keywords") // TODO rename to "Bytes" plugins/discovery-gce/src/main/java/org/elasticsearch/cloud/gce/GceMetadataService.java (1 line): - line 68: // TODO: get this fixed x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/RemoteClusterSecurityRCS1ResolveClusterIT.java (1 line): - line 284: // TODO: The security pathways are not using the new server/src/main/java/org/elasticsearch/action/search/SearchTransportService.java (1 line): - line 407: // TODO: remove this handler once the lowest compatible version stops using it x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/X-ArrayBlock.java.st (1 line): - line 180: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/indices/store/TransportNodesListShardStoreMetadata.java (1 line): - line 159: // TODO: Fallback for BWC with older ES versions. Remove this once request.getCustomDataPath() always returns non-null server/src/main/java/org/elasticsearch/injection/guice/ConstructorInjector.java (1 line): - line 57: // TODO (crazybob): if we can't proxy this object, can we proxy the other object? server/src/main/java/org/elasticsearch/action/admin/cluster/settings/TransportClusterUpdateSettingsAction.java (1 line): - line 263: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/translog/MultiSnapshot.java (1 line): - line 57: // TODO: Read translog forward in 9.0+ x-pack/plugin/sql/sql-proto/src/main/java/org/elasticsearch/xpack/sql/proto/formatter/SimpleFormatter.java (1 line): - line 58: // TODO read the width from the data type? modules/transport-netty4/src/main/java/org/elasticsearch/http/netty4/Netty4HttpContentSizeHandler.java (1 line): - line 48: * TODO: move to RestController to allow content limits per RestHandler. x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/SlmHealthIndicatorService.java (1 line): - line 393: // TODO: handle first snapshot (i.e. no prior success of failure), maybe record the policy first trigger timestamp server/src/main/java/org/elasticsearch/gateway/GatewayService.java (1 line): - line 321: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/fleet/src/main/java/org/elasticsearch/xpack/fleet/Fleet.java (1 line): - line 301: TimeValue.THIRTY_SECONDS /* TODO should we wait longer? */, x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/request/ElasticInferenceServiceRequest.java (1 line): - line 35: // TODO: consider moving tracing here, too x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleBigArrayVector.java (1 line): - line 105: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/indices/recovery/PeerRecoveryTargetService.java (1 line): - line 351: // TODO retries? See RecoveryResponseHandler#handleException server/src/main/java/org/elasticsearch/indices/store/IndicesStore.java (1 line): - line 75: // TODO this class can be foled into either IndicesService and partially into IndicesClusterStateService server/src/main/java/org/elasticsearch/search/aggregations/support/MissingValues.java (1 line): - line 32: // TODO: we could specialize the single value case server/src/main/java/org/elasticsearch/search/suggest/completion/context/CategoryContextMapping.java (1 line): - line 132: // TODO we should be checking mapped field types, not lucene field types x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/TransformPersistentTasksExecutor.java (1 line): - line 262: // TODO: do not use the same error message as for loading the last checkpoint server/src/main/java/org/elasticsearch/index/codec/tsdb/es819/ES819TSDBDocValuesProducer.java (1 line): - line 765: // TODO: should we write to disk the actual max level for this segment? server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/RangeHistogramAggregator.java (1 line): - line 66: // TODO: Stop using nulls here x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/analysis/analyzer/Analyzer.java (1 line): - line 330: // TODO: likely have to expand * inside functions as well server/src/main/java/org/elasticsearch/cluster/service/MasterService.java (1 line): - line 443: // TODO also bubble the failure up to the tasks too, see https://github.com/elastic/elasticsearch/issues/94930 server/src/main/java/org/elasticsearch/action/support/replication/TransportBroadcastReplicationAction.java (1 line): - line 107: // NB This sends O(#shards) requests in a tight loop; TODO add some throttling here? x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elasticsearch/ElasticsearchInternalServiceSettings.java (1 line): - line 89: // model id is optional as the ELSER service will default it. TODO make this a required field once the elser service is removed server/src/main/java/org/elasticsearch/index/termvectors/TermVectorsService.java (1 line): - line 163: // TODO rewrite this to use a field filter built from field patterns server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/InternalVariableWidthHistogram.java (1 line): - line 502: // TODO: Think of a fairer way to do this. Should prev.max = cur.min? x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/support/SecurityIndexManager.java (1 line): - line 449: // TODO we should improve this so we don't fire off a bunch of requests to do the same thing (create or update mappings) server/src/main/java/org/elasticsearch/common/settings/SettingsModule.java (1 line): - line 206: // TODO: remove this hack once we remove the deprecated ability to use repository settings in the cluster state in the S3 snapshot server/src/main/java/org/elasticsearch/action/search/RankFeaturePhase.java (1 line): - line 202: // TODO: handle the exception somewhere x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/async/AsyncTaskIndexService.java (1 line): - line 214: * TODO: add limit for stored async response in EQL, and instead of this method use createResponse x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/categorization/TokenListCategorizer.java (1 line): - line 206: // TODO: improve the calculation of this min and max x-pack/plugin/transform/src/internalClusterTest/java/org/elasticsearch/xpack/transform/integration/TransformNoTransformNodeIT.java (1 line): - line 37: // TODO Change this to run with security enabled libs/x-content/src/main/java/org/elasticsearch/xcontent/XContentParser.java (1 line): - line 205: // TODO remove context entirely when it isn't needed server/src/main/java/org/elasticsearch/search/aggregations/metrics/ScriptedMetricAggregator.java (1 line): - line 91: return ScoreMode.COMPLETE; // TODO: how can we know if the script relies on scores? modules/lang-painless/src/main/java/org/elasticsearch/painless/PainlessScript.java (1 line): - line 69: // TODO: if this is still too long, truncate and use ellipses server/src/internalClusterTest/java/org/elasticsearch/search/query/ExistsIT.java (1 line): - line 39: // TODO: move this to a unit test somewhere... x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/action/TransformPrivilegeChecker.java (1 line): - line 89: // TODO: Remove this filter once https://github.com/elastic/elasticsearch/issues/67798 is fixed. x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/predicate/operator/comparison/InsensitiveEqualsMapper.java (1 line): - line 47: BytesRef rightVal = BytesRefs.toBytesRef(bc.right().fold(FoldContext.small() /* TODO remove me */)); x-pack/plugin/core/src/main/java/org/elasticsearch/snapshots/sourceonly/SourceOnlySnapshotRepository.java (1 line): - line 167: // TODO should we have a snapshot tmp directory per shard that is maintained by the system? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/LongArrayState.java (1 line): - line 113: valuesBuilder.appendLong(0); // TODO can we just use null? modules/mapper-extras/src/main/java/org/elasticsearch/index/mapper/extras/SourceConfirmedTextQuery.java (1 line): - line 391: // TODO what is a right value? libs/core/src/main/java/org/elasticsearch/core/PathUtils.java (1 line): - line 26: // TODO: can we move this to the .env package and make it package-private? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/querylog/EsqlQueryLog.java (1 line): - line 67: return; // TODO review, it happens in some tests, not sure if it's a thing also in prod x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/permission/Role.java (1 line): - line 47: // TODO move once we have a dedicated class for RCS 2.0 constants x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/join/JoinConfig.java (1 line): - line 25: // TODO: this class needs refactoring into a more general form (expressions) since it's currently contains x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/transform/transforms/NodeAttributes.java (1 line): - line 68: // TODO add data_frame attributes when/if they are added server/src/main/java/org/elasticsearch/index/mapper/DotExpandingXContentParser.java (1 line): - line 127: // TODO make this case throw an error instead? https://github.com/elastic/elasticsearch/issues/28948 server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/DoubleTermsIT.java (1 line): - line 106: private static final int NUM_DOCS = 5; // TODO: randomize the size? x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/service/FileServiceAccountTokenStore.java (1 line): - line 169: // TODO: validate against known service accounts? x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/pivot/CompositeBucketsChangeCollector.java (1 line): - line 164: // TODO: based on index.max_terms_count, however this is per index, which we don't have access to here, x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/expression/function/scalar/whitelist/InternalSqlScriptUtils.java (1 line): - line 494: // TODO: Add support for geo_shapes when it is there x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/tokenizers/XLMRobertaTokenizer.java (1 line): - line 43: // TODO Not sure this needs to be a sorted map server/src/main/java/org/elasticsearch/lucene/analysis/miscellaneous/DeDuplicatingTokenFilter.java (1 line): - line 102: // TODO consider changing this implementation to emit tokens as-we-go server/src/main/java/org/elasticsearch/search/vectors/DenseVectorQuery.java (1 line): - line 195: // TODO: can we optimize this at all? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/categorization/InternalCategorizationAggregation.java (1 line): - line 341: // TODO: not sure if this next line is correct - if we discarded some categories due to size or minDocCount is this x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/querydsl/query/MatchQuery.java (1 line): - line 37: // TODO: add zero terms query support, I'm not sure the best way to parse it yet... server/src/main/java/org/elasticsearch/rest/action/admin/cluster/RestNodesInfoAction.java (1 line): - line 97: // disregard unknown metrics; TODO eschew this lenience? server/src/main/java/org/elasticsearch/action/search/DfsQueryPhase.java (1 line): - line 77: // TODO we can potentially also consume the actual per shard results from the initial phase here in the aggregateDfs benchmarks/src/main/java/org/elasticsearch/benchmark/compute/operator/AggregatorBenchmark.java (1 line): - line 79: BigArrays.NON_RECYCLING_INSTANCE // TODO real big arrays? libs/simdvec/src/main22/java/org/elasticsearch/simdvec/internal/Int7SQVectorScorer.java (1 line): - line 52: // TODO assert scalarQuantizer.getBits() == 7 or 8 ? server/src/main/java/org/elasticsearch/action/admin/indices/dangling/delete/TransportDeleteDanglingIndexAction.java (1 line): - line 123: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/action/admin/cluster/stats/CCSUsageTelemetry.java (1 line): - line 134: // TODO: fork this to a background thread? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/process/autodetect/writer/AbstractDataToProcessWriter.java (1 line): - line 270: List headerList = Arrays.asList(header); // TODO header could be empty x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/JobManager.java (1 line): - line 145: // TODO JIndex we shouldn't be building the job here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/UnsupportedAttribute.java (1 line): - line 60: private final boolean hasCustomMessage; // TODO remove me and just use message != null? x-pack/plugin/enrich/src/main/java/org/elasticsearch/xpack/enrich/EnrichStore.java (1 line): - line 198: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here benchmarks/src/main/java/org/elasticsearch/benchmark/compute/operator/TopNBenchmark.java (1 line): - line 54: private static final BigArrays BIG_ARRAYS = BigArrays.NON_RECYCLING_INSTANCE; // TODO real big arrays? x-pack/plugin/esql/compute/gen/src/main/java/org/elasticsearch/compute/gen/ConvertEvaluatorImplementer.java (1 line): - line 72: // TODO support more function types when we need them libs/entitlement/src/main23/java/org/elasticsearch/entitlement/runtime/api/Java23ElasticsearchEntitlementChecker.java (1 line): - line 23: // TODO: this is just an example, we shouldn't really override a method implemented in the superclass x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/job/process/autodetect/state/DataCounts.java (1 line): - line 156: // TODO: Use java.time for the Dates here: x-pack-elasticsearch#829 server/src/main/java/org/elasticsearch/search/aggregations/InternalOrder.java (1 line): - line 39: // TODO merge the contents of this file into BucketOrder. The way it is now is relic. server/src/main/java/org/elasticsearch/persistent/PersistentTasksService.java (1 line): - line 110: // TODO set timeout? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/permission/DocumentPermissions.java (1 line): - line 137: // TODO: All role queries can be filters server/src/main/java/org/elasticsearch/indices/analysis/AnalysisModule.java (1 line): - line 277: // TODO: pluggability? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/ingest/InferenceProcessor.java (1 line): - line 504: // TODO Remove in 9?.x plugins/repository-hdfs/src/main/java/org/elasticsearch/repositories/hdfs/HdfsBlobContainer.java (1 line): - line 73: // TODO: See if we can get precise result reporting. x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportRemoveIndexLifecyclePolicyAction.java (1 line): - line 88: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/action/TransportDeleteSnapshotLifecycleAction.java (1 line): - line 121: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/codec/perfield/XPerFieldDocValuesFormat.java (1 line): - line 210: // TODO: we should only provide the "slice" of FIS server/src/main/java/org/elasticsearch/transport/SniffConnectionStrategy.java (1 line): - line 121: private static final TimeValue SNIFF_REQUEST_TIMEOUT = TimeValue.THIRTY_SECONDS; // TODO make configurable? x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/AuthorizationUtils.java (1 line): - line 169: case TASKS_ORIGIN: // TODO use a more limited user for tasks server/src/main/java/org/elasticsearch/plugins/scanners/ExtensibleFileReader.java (1 line): - line 39: // TODO should we validate the classes actually exist? server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/BytesKeyedBucketOrds.java (1 line): - line 139: // TODO we can almost certainly do better here by building something fit for purpose rather than trying to lego together stuff server/src/main/java/org/elasticsearch/action/admin/indices/forcemerge/TransportForceMergeAction.java (1 line): - line 99: assert (task instanceof CancellableTask) == false; // TODO: add cancellation handling here once the task supports it distribution/tools/server-cli/src/main/java/org/elasticsearch/server/cli/ServerCli.java (1 line): - line 280: // TODO: Use the environment configuration to decide what kind of secrets store to load x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/privilege/ApplicationPrivilege.java (1 line): - line 51: // TODO make this private once ApplicationPrivilegeTests::createPrivilege uses ApplicationPrivilege::get server/src/main/java/org/elasticsearch/cluster/ClusterModule.java (1 line): - line 421: // TODO: this is public so allocation benchmark can access the default deciders...can we do that in another way? server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/SignificanceLookup.java (1 line): - line 241: // TODO this method helps because of asMultiBucketAggregator. Once we remove it we can move this logic into the aggregators. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/support/Automatons.java (1 line): - line 402: // TODO: cutover to iterators instead x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/CompositeBlock.java (1 line): - line 189: // TODO: support this server/src/main/java/org/elasticsearch/action/admin/cluster/stats/LongMetric.java (1 line): - line 75: // TODO: not sure what is the good value for minBarForHighestToLowestValueRatio here? modules/data-streams/src/main/java/org/elasticsearch/datastreams/action/TransportPromoteDataStreamAction.java (1 line): - line 95: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/FilteredAggregatorFunction.java (1 line): - line 34: // TODO warn on mv fields x-pack/plugin/downsample/src/main/java/org/elasticsearch/xpack/downsample/FieldValueFetcher.java (1 line): - line 69: // TODO: Support POSITION in downsampling x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/kstest/BucketCountKSTestAggregator.java (1 line): - line 85: // TODO bootstrap to support sparse data client/rest/src/main/java/org/elasticsearch/client/RestClient.java (1 line): - line 241: // TODO should we throw an IAE if we have two nodes with the same host? server/src/main/java/org/elasticsearch/injection/guice/LookupProcessor.java (1 line): - line 41: errors.merge(e.getErrors()); // TODO: source x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleLookup.java (1 line): - line 86: // TODO replace this with a warning and break server/src/main/java/org/elasticsearch/index/mapper/MappedFieldType.java (1 line): - line 236: // TODO: Standardize exception types x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/watcher/condition/Condition.java (1 line): - line 43: // TODO: FAILURE status is never used, but a some code assumes that it is used x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/ClusterComputeRequest.java (1 line): - line 67: // TODO make EsqlConfiguration Releasable server/src/main/java/org/elasticsearch/action/support/TransportAction.java (1 line): - line 181: // TODO remove this when https://github.com/elastic/elasticsearch/issues/100111 is resolved x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/AggregationOperator.java (1 line): - line 142: // TODO: look into allocating the blocks lazily x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/process/autodetect/AutodetectProcessManager.java (1 line): - line 740: // TODO: in future this will also need to consider jobs that are not anomaly detector jobs server/src/main/java/org/elasticsearch/search/aggregations/pipeline/MovingFunctions.java (1 line): - line 288: // TODO if perf is a problem, we can specialize a subclass to avoid conditionals on each iteration server/src/main/java/org/elasticsearch/search/aggregations/support/ValuesSourceConfig.java (1 line): - line 303: // TODO: resolve no longer generates invalid configs. Once VSConfig is immutable, we can drop this check server/src/main/java/org/elasticsearch/rest/action/document/RestBulkAction.java (1 line): - line 142: // TODO: Maybe improve in follow-up to be XContentParseException and include line number and column x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/MedianAbsoluteDeviation.java (1 line): - line 40: // TODO: Add parameter server/src/main/java/org/elasticsearch/search/profile/AbstractInternalProfileTree.java (1 line): - line 143: // TODO this would be better done bottom-up instead of top-down to avoid server/src/main/java/org/elasticsearch/index/codec/bloomfilter/ES85BloomFilterPostingsFormat.java (1 line): - line 297: // TODO: return TermState that includes BloomFilter and fix _disk_usage API server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/NumericHistogramAggregator.java (1 line): - line 71: // TODO: Stop using null here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/execution/PlanExecutor.java (1 line): - line 113: // TODO when we decide if we will differentiate Kibana from REST, this String value will likely come from the request server/src/main/java/org/elasticsearch/common/util/concurrent/AbstractThrottledTaskRunner.java (1 line): - line 71: protected boolean isForceExecution(@SuppressWarnings("unused") /* TODO test this */ T task) { x-pack/plugin/security/src/internalClusterTest/java/org/elasticsearch/xpack/security/authz/store/NativePrivilegeStoreCacheTests.java (1 line): - line 268: // TODO: This is a bug x-pack/plugin/downsample/src/main/java/org/elasticsearch/xpack/downsample/DownsampleShardPersistentTaskExecutor.java (1 line): - line 315: // TODO: consider moving to Downsample.DOWSAMPLE_TASK_THREAD_POOL_NAME and simplify realNodeOperation server/src/internalClusterTest/java/org/elasticsearch/search/basic/SearchWhileCreatingIndexIT.java (1 line): - line 45: // TODO: randomize the wait for active shards value on index creation and ensure the appropriate server/src/main/java/org/elasticsearch/index/mapper/ObjectMapper.java (1 line): - line 566: // TODO stop NestedObjectMapper extending ObjectMapper? x-pack/plugin/transform/src/internalClusterTest/java/org/elasticsearch/xpack/transform/integration/TransformOldTransformsIT.java (1 line): - line 48: // TODO Change this to run with security enabled x-pack/plugin/migrate/src/main/java/org/elasticsearch/xpack/migrate/action/ReindexDataStreamTransportAction.java (1 line): - line 123: TimeValue.THIRTY_SECONDS /* TODO should this be configurable? longer by default? infinite? */, server/src/main/java/org/elasticsearch/action/search/CCSSingleCoordinatorSearchProgressListener.java (1 line): - line 154: // TODO in the fail-fast ticket, should we throw an exception here to stop the search? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/RateIntAggregator.java (1 line): - line 176: // TODO: add bulk_copy to Block x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/RLike.java (1 line): - line 117: // TODO: see whether escaping is needed modules/parent-join/src/main/java/org/elasticsearch/join/query/HasChildQueryBuilder.java (1 line): - line 370: // TODO: Find a way to remove this query and let doToQuery(...) just return the query from JoinUtil.createJoinQuery(...) modules/aggregations/src/main/java/org/elasticsearch/aggregations/pipeline/BucketSelectorPipelineAggregator.java (1 line): - line 66: // TODO: can we use one instance of the script for all buckets? it should be stateless? x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/expression/Expression.java (1 line): - line 93: // TODO After removing FoldContext.unbounded from non-test code examine all calls x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/function/scalar/ScalarFunction.java (1 line): - line 80: // FIXME: this needs to be refactored x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/assignment/TrainedModelAssignment.java (1 line): - line 41: // TODO implement better diffable logic so that whole diff does not need to be serialized if only one part changes x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/ConstantDoubleVector.java (1 line): - line 62: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/TrainedModelConfig.java (1 line): - line 419: // TODO if we ever support anything other than "basic" and platinum, we need to adjust our feature tracking logic x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/BytesRef3BlockHash.java (1 line): - line 150: // TODO Build Ordinals blocks #114010 x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/CountDistinct.java (1 line): - line 215: : ((Number) this.precision.fold(FoldContext.small() /* TODO remove me */)).intValue(); x-pack/plugin/enrich/src/internalClusterTest/java/org/elasticsearch/xpack/enrich/EnrichProcessorIT.java (1 line): - line 43: // TODO Change this to run with security enabled server/src/main/java/org/elasticsearch/action/termvectors/TermVectorsRequest.java (1 line): - line 83: // TODO: change to String[] x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/GeneratingPlan.java (1 line): - line 29: // TODO: the generated attributes should probably become synthetic once renamed server/src/main/java/org/elasticsearch/search/internal/ShardSearchContextId.java (1 line): - line 24: // TODO: Remove this constructor server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/InternalSignificantTerms.java (1 line): - line 90: // TODO we should refactor to remove this, since buckets should be immutable after they are generated. x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/EsqlPlugin.java (1 line): - line 325: // TODO: Maybe have two types of threadpools for workers: one for CPU-bound and one for I/O-bound tasks. server/src/main/java/org/elasticsearch/common/blobstore/BlobStoreActionStats.java (1 line): - line 36: // TODO: assert that requests >= operations once https://elasticco.atlassian.net/browse/ES-10223 is played server/src/internalClusterTest/java/org/elasticsearch/search/sort/SimpleSortIT.java (1 line): - line 333: // TODO: sort shouldn't fail when sort field is mapped dynamically x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntArrayBlock.java (1 line): - line 134: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/index/mapper/GeoPointFieldMapper.java (1 line): - line 311: // TODO phase out geohash (which is currently used in the CompletionSuggester) server/src/main/java/org/elasticsearch/cluster/metadata/MetadataMigrateToDataStreamService.java (1 line): - line 137: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/permission/FieldPermissions.java (1 line): - line 205: // TODO: cache the automaton computation with FieldPermissionsCache x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/mapper/MapperUtils.java (1 line): - line 116: BytesRefs.toString(enrich.policyName().fold(FoldContext.small() /* TODO remove me */)), libs/geo/src/main/java/org/elasticsearch/geometry/simplify/GeometrySimplifier.java (1 line): - line 356: // TODO: Find a way to get this method to return specialized simplifiers for non-identity cases (eg. Line and Polygon) x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/exchange/ExchangeRequest.java (1 line): - line 47: // TODO: Maybe add a separate action/request for closing exchange sinks? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/datafeed/DatafeedJobValidator.java (1 line): - line 34: // TODO should we validate that the aggs define the fields requested in the analysis config? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/TimeSeriesAggregationOperator.java (1 line): - line 42: // TODO: use TimeSeriesBlockHash when possible x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/watcher/support/WatcherDateTimeUtils.java (1 line): - line 176: // TODO we can potentially remove this in 6.x server/src/main/java/org/elasticsearch/search/aggregations/metrics/NumericMetricsAggregator.java (1 line): - line 105: // TODO it'd be faster replace hasMetric and metric with something that returned a function from long to double. server/src/main/java/org/elasticsearch/index/mapper/BinaryRangeUtil.java (1 line): - line 133: // TODO: Support for exclusive ranges, pending resolution of #40601 server/src/main/java/org/elasticsearch/search/SearchHit.java (1 line): - line 880: // TODO: can we avoid having an exception here? server/src/main/java/org/elasticsearch/search/aggregations/bucket/terms/RareTermsAggregatorFactory.java (1 line): - line 61: ExecutionMode execution = ExecutionMode.MAP; // TODO global ords not implemented yet, only supports "map" benchmarks/src/main/java/org/elasticsearch/benchmark/compute/operator/BlockReadBenchmark.java (1 line): - line 106: // TODO float x-pack/plugin/enrich/src/main/java/org/elasticsearch/xpack/enrich/EnrichPolicyReindexPipeline.java (1 line): - line 40: * TODO: This should be the version of ES that Enrich first ships in, which likely doesn't exist yet. x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/MlIndexTemplateRegistry.java (1 line): - line 44: * 10000001: TODO - reason x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/evaluator/command/GrokEvaluatorExtracter.java (1 line): - line 178: // TODO remove when floats are supported x-pack/plugin/esql/compute/gen/src/main/java/org/elasticsearch/compute/gen/GroupingAggregatorImplementer.java (1 line): - line 119: // TODO support multiple parameters x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/termsenum/action/MultiShardTermsEnum.java (1 line): - line 116: // TODO: we could maybe defer this somewhat costly operation until one of the APIs that server/src/main/java/org/elasticsearch/index/fielddata/NumericDoubleValues.java (1 line): - line 26: // TODO: this interaction with sort comparators is really ugly... server/src/main/java/org/elasticsearch/search/aggregations/metrics/MetricInspectionHelper.java (1 line): - line 40: // TODO better way to know if the scripted metric received documents? x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elasticsearch/BaseElasticsearchInternalService.java (1 line): - line 71: // TODO make package private once the elser service is moved to the Elasticsearch x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/store/CompositeRolesStore.java (1 line): - line 610: false, // TODO revisit if we should also wait for an available security index here modules/transport-netty4/src/main/plugin-metadata/entitlement-policy.yaml (1 line): - line 7: # TODO: copied over from the security policy. Check if this is still valid x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/BWCCodec.java (1 line): - line 200: // TODO: perhaps store the original version information in attributes so that we can retrieve it later when needed? server/src/main/java/org/elasticsearch/indices/IndicesService.java (1 line): - line 1929: // TODO move this? server/src/main/java/org/elasticsearch/common/util/BigArrays.java (1 line): - line 560: public CircuitBreakerService breakerService() { // TODO this feels like it is for tests but it has escaped server/src/main/java/org/elasticsearch/cluster/routing/DelayedAllocationService.java (1 line): - line 128: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/cohere/response/CohereRankedResponseEntity.java (1 line): - line 95: positionParserAtTokenAfterField(jsonParser, "results", FAILED_TO_FIND_FIELD_TEMPLATE); // TODO error message build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/doc/DocsTestPlugin.java (1 line): - line 103: // TODO: This effectively makes testRoot not customizable, which we don't do anyway atm benchmarks/src/main/java/org/elasticsearch/benchmark/compute/operator/ValuesAggregatorBenchmark.java (1 line): - line 83: BigArrays.NON_RECYCLING_INSTANCE // TODO real big arrays? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/deployment/InferencePyTorchAction.java (1 line): - line 117: // TODO batching was implemented for expected use-cases such as zero-shot classification but is not used here. x-pack/license-tools/src/main/java/org/elasticsearch/license/licensor/tools/LicenseGeneratorTool.java (1 line): - line 43: // TODO: with jopt-simple 5.0, we can make these requiredUnless each other x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/FilteredGroupingAggregatorFunction.java (1 line): - line 45: // TODO warn on mv fields x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/OrdinalsGroupingOperator.java (1 line): - line 238: // TODO: Should we also combine from the results from ValuesAggregator x-pack/plugin/sql/sql-client/src/main/java/org/elasticsearch/xpack/sql/client/UriUtils.java (1 line): - line 39: // TODO: support for IDN x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/search/aggregations/metrics/GeoShapeCentroidAggregator.java (1 line): - line 34: * TODO: This can be generalized with the nearly identical class CartesianShapeCentroidAggregator x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/xpack/searchablesnapshots/store/input/DirectBlobContainerIndexInput.java (1 line): - line 103: super(name, bufferSize, length); // TODO should use blob cache server/src/main/java/org/elasticsearch/search/aggregations/bucket/sampler/random/RandomSamplerAggregationBuilder.java (1 line): - line 134: // TODO add a method or interface to aggregation builder that defaults to false server/src/main/java/org/elasticsearch/search/aggregations/bucket/sampler/random/RandomSamplerAggregator.java (1 line): - line 139: // TODO know when sampling would be much slower and skip sampling: https://github.com/elastic/elasticsearch/issues/84353 x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/MachineLearningUsageTransportAction.java (1 line): - line 588: // TODO separate out ours and users models possibly regression vs classification x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/indexing/AsyncTwoPhaseIndexer.java (1 line): - line 554: // TODO we should check items in the response and move after accordingly to x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/DoubleArrayBlock.java (1 line): - line 134: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportDeleteTrainedModelAliasAction.java (1 line): - line 87: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/resources/org/elasticsearch/bootstrap/untrusted.policy (1 line): - line 16: // TODO: clean this up / only give it to engines that really must have it x-pack/license-tools/src/main/java/org/elasticsearch/license/licensor/tools/KeyPairGeneratorTool.java (1 line): - line 35: // TODO: in jopt-simple 5.0 we can use a PathConverter to take Path instead of File x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvConcat.java (1 line): - line 149: BytesRefBuilder work = new BytesRefBuilder(); // TODO BreakingBytesRefBuilder so we don’t blow past circuit breakers modules/data-streams/src/internalClusterTest/java/org/elasticsearch/datastreams/DataStreamIT.java (1 line): - line 326: // TODO: remove when fixing the bug when an index matching a backing index name is created before the data stream is created x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/action/TransportUpgradeTransformsAction.java (1 line): - line 151: * TODO: Handle deprecated data_frame_transform roles x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/ConvertStringToByteRef.java (1 line): - line 26: // TODO we shouldn't be emitting String into Literals at all x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/googlevertexai/GoogleVertexAiService.java (1 line): - line 336: // TODO whether the model ID is required or not depends on the task type x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/PkiRealmBootstrapCheck.java (1 line): - line 77: // FIXME this is an antipattern move this out of a bootstrap check! server/src/main/java/org/elasticsearch/action/search/ExpandSearchPhase.java (1 line): - line 127: // TODO: make this work pooled by forcing the hit itself to become pooled as needed here modules/lang-expression/src/internalClusterTest/java/org/elasticsearch/script/expression/StoredExpressionIT.java (1 line): - line 27: //TODO: please convert to unit tests! x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/InlineStats.java (1 line): - line 97: // TODO: in case of inlinestats, the join key is always the grouping server/src/main/java/org/elasticsearch/action/admin/indices/mapping/get/TransportGetFieldMappingsIndexAction.java (1 line): - line 159: // TODO the logic here needs to be reworked to also include runtime fields. Though matching is against mappers rather x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/util/DateUtils.java (1 line): - line 31: //FIXME: Taken from sql-proto (StringUtils) x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/deployment/DeploymentManager.java (1 line): - line 588: priorityProcessWorker.shutdownNow(); // TODO what to do with these tasks? x-pack/plugin/esql/compute/gen/src/main/java/org/elasticsearch/compute/gen/AggregatorImplementer.java (1 line): - line 110: // TODO support multiple parameters x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/transform/script/ExecutableScriptTransform.java (1 line): - line 50: // TODO: deprecate one of these styles (returning a map or returning an opaque value below) server/src/main/java/org/elasticsearch/bootstrap/BootstrapSettings.java (1 line): - line 19: // TODO: remove this hack when insecure defaults are removed from java x-pack/plugin/sql/qa/server/src/main/java/org/elasticsearch/xpack/sql/qa/jdbc/SpecBaseIntegrationTestCase.java (1 line): - line 115: // TODO: use UTC for now until deciding on a strategy for handling date extraction server/src/main/java/org/elasticsearch/search/suggest/phrase/CandidateGenerator.java (1 line): - line 18: //TODO public for tests x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/type/StringUtils.java (1 line): - line 27: //FIXME: this class comes from sql-proto x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/Count.java (1 line): - line 168: // TODO: Normalize COUNT(*), COUNT(), COUNT("foobar"), COUNT(1) as COUNT(*). server/src/main/java/org/elasticsearch/index/fieldvisitor/FieldsVisitor.java (1 line): - line 93: continue; // TODO this is lame x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongBigArrayVector.java (1 line): - line 105: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/condition/WatcherConditionScript.java (1 line): - line 24: // TODO: ctx should have its members extracted into execute parameters, but it needs to be a member for bwc access in params modules/repository-gcs/src/main/java/org/elasticsearch/repositories/gcs/GoogleCloudStorageRetryingInputStream.java (1 line): - line 213: // TODO: check that object did not change when stream is reopened (e.g. based on etag) x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/planner/QueryTranslator.java (1 line): - line 267: // TODO: see whether escaping is needed x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/azureopenai/AzureOpenAiModel.java (1 line): - line 98: // TODO: can be inferred directly from modelConfigurations.getServiceSettings(); will be addressed with separate refactoring x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/CombineEvals.java (1 line): - line 16: * TODO: eliminate unnecessary fields inside the eval as well x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/ForwardBytesReader.java (1 line): - line 22: // TODO: can we use just ByteArrayDataInput...? need to qa/lucene-index-compatibility/src/javaRestTest/java/org/elasticsearch/lucene/AbstractIndexCompatibilityTestCase.java (1 line): - line 111: // TODO ES-10475: The .snapshot-blob-cache created in legacy version can block upgrades, we should probably delete it automatically x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/service/CompositeServiceAccountTokenStore.java (1 line): - line 35: // TODO: optimize store order based on auth result? x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/job/RollupIndexer.java (1 line): - line 299: // TODO allow non-numeric value_counts. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/search/TextExpansionQueryBuilder.java (1 line): - line 221: // TODO this should be updated to always use a WeightedTokensQueryBuilder once it's in all supported versions. server/src/main/java/org/elasticsearch/index/search/SimpleQueryStringQueryParser.java (1 line): - line 247: // TODO: we should not ignore the exception and return a prefix query with the original term ? server/src/main/java/org/elasticsearch/index/SlowLogLevel.java (1 line): - line 16: * TODO: Remove in 9.0 x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/index/fielddata/CartesianShapeValues.java (1 line): - line 87: * TODO: this is test only method, perhaps should be moved to test code x-pack/plugin/inference/src/internalClusterTest/java/org/elasticsearch/xpack/inference/action/filter/ShardBulkInferenceActionFilterBasicLicenseIT.java (1 line): - line 68: // TODO: Allow element type BIT once TestDenseInferenceServiceExtension supports it server/src/main/java/org/elasticsearch/search/aggregations/bucket/missing/MissingAggregator.java (1 line): - line 43: // TODO: Stop using nulls here x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/topmetrics/TopMetricsAggregationBuilder.java (1 line): - line 115: // TODO replace with ValuesSourceConfig once the value source refactor has landed x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/TransportEsqlGetQueryAction.java (1 line): - line 96: // TODO include completed drivers in documentsFound and valuesLoaded server/src/main/java/org/elasticsearch/watcher/FileWatcher.java (1 line): - line 153: // TODO we might use the new NIO2 API to get real notification? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/type/SqlDataTypes.java (1 line): - line 707: // TODO: return info for SCALED_FLOATS (should be based on field not type) build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/precommit/ForbiddenPatternsTask.java (1 line): - line 167: // TODO: fail if pattern contains a newline, it won't work (currently) server/src/main/java/org/elasticsearch/search/aggregations/package-info.java (1 line): - line 29: *

TODO: Info about search phases goes here

x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/assignment/AssignmentStats.java (1 line): - line 595: // TODO: consider adding another field here for inference cache hits, but mindful of the naming collision x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/ZeroShotClassificationProcessor.java (1 line): - line 145: // TODO only the first entry in the batch result is verified and x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/tokenizers/RobertaTokenizer.java (1 line): - line 41: // TODO Not sure this needs to be a sorted map x-pack/plugin/sql/qa/jdbc/src/main/java/org/elasticsearch/xpack/sql/qa/jdbc/ResultSetTestCase.java (1 line): - line 1035: // TODO: no half & scaled float testing x-pack/plugin/transform/src/internalClusterTest/java/org/elasticsearch/xpack/transform/integration/TransformNoRemoteClusterClientNodeIT.java (1 line): - line 35: // TODO Change this to run with security enabled server/src/main/java/org/elasticsearch/search/suggest/phrase/MultiCandidateGeneratorWrapper.java (1 line): - line 19: //TODO public for tests x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/fulltext/Match.java (1 line): - line 435: Object queryAsObject = query().fold(FoldContext.small() /* TODO remove me */); x-pack/plugin/vector-tile/src/main/java/org/elasticsearch/xpack/vectortile/rest/VectorTileRequest.java (1 line): - line 309: // TODO: validation server/src/internalClusterTest/java/org/elasticsearch/search/sort/FieldSortIT.java (1 line): - line 982: // TODO: WTF? server/src/main/java/org/elasticsearch/index/IndexingSlowLog.java (1 line): - line 80: * TODO: Remove in 9.0 x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/expression/function/scalar/geo/UnaryGeoFunction.java (1 line): - line 57: // TODO change this to use _source instead of the exact form (aka field.keyword for geo shape fields) x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/formatter/TextFormatter.java (1 line): - line 48: // TODO read the width from the data type? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/annotations/AnnotationIndex.java (1 line): - line 115: .prepareAliases(masterNodeTimeout, TimeValue.THIRTY_SECONDS) // TODO does acking matter? If so, should we wait longer? server/src/main/java/org/elasticsearch/cluster/node/VersionInformation.java (1 line): - line 64: // TODO: work out what needs to happen for other versions. Maybe we can only remove this once the nodeVersion field is gone server/src/main/java/org/elasticsearch/script/ScriptContextInfo.java (1 line): - line 332: // TODO: ensure empty/no PARAMETERS if parameterTypes.length == 0? server/src/internalClusterTest/java/org/elasticsearch/snapshots/SharedClusterSnapshotRestoreIT.java (1 line): - line 1929: // TODO: Fix repo cleanup logic to handle these leaked snap-file and only exclude test-repo (the mock repo) here. x-pack/plugin/eql/qa/security/src/javaRestTest/java/org/elasticsearch/xpack/eql/EqlRestValidationIT.java (1 line): - line 53: // TODO: revisit the next two tests when https://github.com/elastic/elasticsearch/issues/64190 is closed x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/tree/Node.java (1 line): - line 173: // TODO: maybe add a flatMap (need to double check the Stream bit) x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/CrossClusterEsqlRCS1MissingIndicesIT.java (1 line): - line 238: // TODO: in follow on PR, add support for throwing a VerificationException from this scenario x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/SnapshotLifecycleService.java (1 line): - line 109: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/gateway/PersistedClusterStateService.java (1 line): - line 1259: // TODO multi-project: we might want to write each project as a separate document instead of one huge document. server/src/main/java/org/elasticsearch/synonyms/SynonymRule.java (1 line): - line 143: // TODO: should we use String::split instead? x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/rate/TimeSeriesRateAggregator.java (1 line): - line 114: leafValues.advanceExact(doc); // TODO handle missing values x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/ShardFollowNodeTask.java (1 line): - line 270: // TODO: To ensure that we never overuse the buffer, we need to x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/huggingface/elser/HuggingFaceElserService.java (1 line): - line 110: // TODO chunking sparse embeddings not implemented x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/ScoreOperator.java (1 line): - line 67: // TODO Optimize for constant scores? x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/tree/Node.java (1 line): - line 164: // TODO: maybe add a flatMap (need to double check the Stream bit) x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/X-Lookup.java.st (1 line): - line 97: // TODO replace this with a warning and break x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/CombineProjections.java (1 line): - line 169: // TODO: The deduplication based on simple equality will be insufficient in case of multiple non-evaluatable groupings, e.g. for x-pack/plugin/sql/qa/server/single-node/src/javaRestTest/java/org/elasticsearch/xpack/sql/qa/single_node/CliExplainIT.java (1 line): - line 49: // TODO in this case we should probably remove the source filtering entirely. Right? It costs but we don't need it. server/src/main/java/org/elasticsearch/gateway/LocalAllocateDangledIndices.java (1 line): - line 249: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/codec/vectors/es818/DirectIOLucene99FlatVectorsFormat.java (1 line): - line 68: // TODO: Open the mmap file with sequential access instead of random (current behavior). x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/TextClassificationProcessor.java (1 line): - line 109: // TODO adjust logic when batch is allowed server/src/main/java/org/elasticsearch/indices/recovery/RecoverySourceHandler.java (1 line): - line 361: final long phase1ThrottlingWaitTime = 0L; // TODO: return the actual throttle time x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/ExchangeSinkExec.java (1 line): - line 30: // TODO: remove this flag server/src/main/java/org/elasticsearch/client/internal/Requests.java (1 line): - line 22: * TODO: remove this, we shouldn't have mutable public static fields that we use in prod code x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/process/MlControllerHolder.java (1 line): - line 16: * TODO: remove this class entirely once Guice is removed entirely. server/src/main/java/org/elasticsearch/common/logging/internal/LoggerFactoryImpl.java (1 line): - line 21: // TODO PG logger impl instance caching https://github.com/elastic/elasticsearch/issues/87511 x-pack/plugin/security/src/internalClusterTest/java/org/elasticsearch/integration/KibanaUserRoleIntegTests.java (1 line): - line 181: // TODO: When we have an XPackIntegTestCase, this should test that we can send MonitoringBulkActions x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/rollup/action/RollupJobCaps.java (1 line): - line 52: // TODO now that these rollup caps are being used more widely (e.g. search), perhaps we should x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/planner/ExpressionTranslators.java (1 line): - line 122: // TODO: see whether escaping is needed x-pack/plugin/security/src/internalClusterTest/java/org/elasticsearch/xpack/security/authc/pki/PkiAuthenticationTests.java (1 line): - line 86: // TODO: consider setting this back to true now that the transport client is gone x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/preprocessing/customwordembedding/FeatureUtils.java (1 line): - line 75: // 5. Replace Turkish İ with I (TODO - check this out better...) x-pack/plugin/watcher/src/main/plugin-metadata/entitlement-policy.yaml (1 line): - line 5: # TODO: this is likely not needed, but including here to be on the safe side until x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elasticsearch/ElserMlNodeTaskSettings.java (1 line): - line 65: // TODO Class has no members all instances are equivalent x-pack/plugin/deprecation/src/main/java/org/elasticsearch/xpack/deprecation/IndexDeprecationChecker.java (1 line): - line 96: // TODO: this check needs to be revised. It's trivially true right now. server/src/main/java/org/elasticsearch/action/admin/cluster/configuration/TransportAddVotingConfigExclusionsAction.java (1 line): - line 182: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/engine/MergeMemoryEstimator.java (1 line): - line 90: // TODO Work on estimations on other field infos when / if needed x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/PropagateEquals.java (1 line): - line 60: // TODO: In 105217, this should change to EsqlBinaryComparison, but it doesn't exist in this branch yet x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/util/NumericUtils.java (1 line): - line 122: if (unsignedLongMultiplyHigh(ux, uy) != 0) { // TODO: replace with Math#unsignedMultiplyHigh() in JDK 18 when available server/src/main/java/org/elasticsearch/action/update/TransportUpdateAction.java (1 line): - line 475: // TODO: Fix bug here when semantic text field is in an object modules/repository-s3/src/javaRestTest/java/org/elasticsearch/repositories/s3/RepositoryS3MinioBasicCredentialsRestIT.java (1 line): - line 46: // TODO do not set these if running a MinIO version in which https://github.com/minio/minio/issues/21189 is fixed server/src/main/java/org/elasticsearch/action/ingest/IngestActionForwarder.java (1 line): - line 29: * TODO: move this into IngestService and make index/bulk actions call that server/src/main/java/org/elasticsearch/index/mapper/DocumentParser.java (1 line): - line 853: // TODO: passing null to an object seems bogus? x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene54/Lucene54DocValuesProducer.java (1 line): - line 1117: // TODO: use .nextSetBit here, at least!! server/src/main/java/org/elasticsearch/lucene/queries/BlendedTermQuery.java (1 line): - line 105: // TODO: Maybe it could also make sense to assume independent distributions of documents and eg. have: x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/downsample/DownsampleShardTask.java (1 line): - line 83: // TODO: just for testing x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/EsStatsQueryExec.java (1 line): - line 100: // TODO - get the estimation outside the plan so it doesn't touch the plan x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlResolveFieldsAction.java (1 line): - line 45: // TODO replace DIRECT_EXECUTOR_SERVICE when removing workaround for https://github.com/elastic/elasticsearch/issues/97916 x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportDeleteLifecycleAction.java (1 line): - line 112: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/OrdinalBytesRefBlock.java (1 line): - line 107: // TODO: merge this BytesRefArrayBlock#filter x-pack/plugin/sql/sql-proto/src/main/java/org/elasticsearch/xpack/sql/proto/VersionCompatibility.java (1 line): - line 41: // TODO: move to VersionCompatibilityChecks server/src/main/java/org/elasticsearch/cluster/metadata/MetadataDeleteIndexService.java (1 line): - line 153: // TODO Avoid creating the state multiple times if there are batched updates for multiple projects x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/math/Round.java (1 line): - line 50: // @TODO: add support for "integer", "long", "unsigned_long" once tests are fixed modules/reindex/src/main/java/org/elasticsearch/reindex/BulkByScrollParallelizationHelper.java (1 line): - line 168: // TODO move the request to the correct node. maybe here or somehow do it as part of startup for reindex in general.... x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/action/GetJobsStatsAction.java (1 line): - line 239: // TODO: Have callers wrap the content with an object as they choose rather than forcing it upon them x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/datafeed/extractor/DataExtractorFactory.java (1 line): - line 89: // TODO Rollup V2 will support runtime fields server/src/main/java/org/elasticsearch/search/suggest/phrase/WordScorer.java (1 line): - line 22: //TODO public for tests x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportPutDataFrameAnalyticsAction.java (1 line): - line 163: // TODO: Remove this filter once https://github.com/elastic/elasticsearch/issues/67798 is fixed. server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/TermsDocCountErrorIT.java (1 line): - line 58: // TODO: it's practically impossible to get a 100% deterministic test with batched execution unfortunately, adjust this test to x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/profile/ProfileService.java (1 line): - line 173: // TODO: with request when we take request body for profile activation server/src/main/java/org/elasticsearch/common/util/CombinedRateLimiter.java (1 line): - line 22: // TODO: This rate limiter has some concurrency issues between the two maybePause operations x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/LookupJoinTypesIT.java (1 line): - line 170: // TODO: We should probably allow this, but we need to change the validation code in Join.java server/src/main/java/org/elasticsearch/cluster/routing/allocation/DiskThresholdMonitor.java (1 line): - line 146: // TODO find a better way to limit concurrent updates (and potential associated reroutes) while allowing tests to ensure that x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/ServiceUtils.java (1 line): - line 192: // TODO map as JSON x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/execution/search/Querier.java (1 line): - line 103: // TODO: add retry/back-off server/src/main/java/org/elasticsearch/rest/RestHandler.java (1 line): - line 94: * TODO - make this not nullable when all handlers have been updated x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/querydsl/query/MultiMatchQuery.java (1 line): - line 46: // TODO: add zero terms query support, I'm not sure the best way to parse it yet... x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/planner/AggregateMapper.java (1 line): - line 35: // TODO: Do we need this cache? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/FloatBigArrayVector.java (1 line): - line 93: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/search/aggregations/AggregationExecutionContext.java (1 line): - line 27: private final Supplier tsidProvider; // TODO remove this entirely? server/src/main/java/org/elasticsearch/index/search/MultiMatchQueryParser.java (1 line): - line 307: // TODO: can we improve this? server/src/main/java/org/elasticsearch/search/suggest/phrase/Correction.java (1 line): - line 17: //TODO public for tests x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/IntArrayState.java (1 line): - line 107: valuesBuilder.appendInt(0); // TODO can we just use null? modules/transport-netty4/src/main/java/org/elasticsearch/transport/netty4/Netty4TcpChannel.java (1 line): - line 67: // TODO: handle this cleaner? server/src/main/java/org/elasticsearch/action/admin/indices/flush/TransportShardFlushAction.java (1 line): - line 100: // TODO: Remove this transition in 9.0 x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/datafeed/DatafeedManager.java (1 line): - line 108: // TODO: Remove this filter once https://github.com/elastic/elasticsearch/issues/67798 is fixed. build-conventions/src/main/java/org/elasticsearch/gradle/internal/conventions/EclipseConventionPlugin.java (1 line): - line 86: // TODO: "package this up" for external builds x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/Function.java (1 line): - line 71: * TODO: replace the boolean with a more descriptive enum. x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/expression/TypeResolutions.java (1 line): - line 214: // TODO: Shouldn't we perform widening of small numerical types here? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authz/permission/LimitedRole.java (1 line): - line 60: // TODO: this is to retain existing behaviour, but it is not accurate libs/entitlement/src/main/java/org/elasticsearch/entitlement/runtime/policy/PolicyManager.java (1 line): - line 598: // TODO: Use a more human-readable description. Perhaps share code with InstrumentationServiceImpl.parseCheckerMethodName x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportRetryAction.java (1 line): - line 101: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/mapper/AbstractScriptFieldType.java (1 line): - line 231: // TODO rework things so that we don't need this server/src/internalClusterTest/java/org/elasticsearch/cluster/routing/ShardRoutingRoleIT.java (1 line): - line 643: // TODO add support for ONLY_LOCAL and ONLY_NODES too server/src/main/java/org/elasticsearch/script/ScriptService.java (1 line): - line 762: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/search/aggregations/bucket/sampler/SamplerAggregator.java (1 line): - line 36: * TODO currently the diversity feature of this agg offers only 'script' and x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanLookup.java (1 line): - line 86: // TODO replace this with a warning and break server/src/main/java/org/elasticsearch/common/xcontent/SuggestingErrorOnUnknown.java (1 line): - line 37: // TODO it'd be nice to combine this with BaseRestHandler's implementation. server/src/internalClusterTest/java/org/elasticsearch/indices/memory/breaker/RandomExceptionCircuitBreakerIT.java (1 line): - line 199: // TODO: Generalize this class and add it as a utility server/src/main/java/org/elasticsearch/search/query/QuerySearchResult.java (1 line): - line 132: * TODO: Currently we need the concrete aggregators to build empty responses. This means that we cannot x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/ibmwatsonx/response/IbmWatsonxRankedResponseEntity.java (1 line): - line 87: positionParserAtTokenAfterField(jsonParser, "results", FAILED_TO_FIND_FIELD_TEMPLATE); // TODO error message x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/movingPercentiles/MovingPercentilesPipelineAggregator.java (1 line): - line 264: // TODO: replace this with the PercentilesConfig that's used by the percentiles builder. x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/ProjectExec.java (1 line): - line 23: public class ProjectExec extends UnaryExec { // TODO implement EstimatesRowSize *somehow* x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/Explain.java (1 line): - line 45: // TODO: implement again x-pack/plugin/rollup/src/main/java/org/elasticsearch/xpack/rollup/action/TransportPutRollupJobAction.java (1 line): - line 326: TimeValue.THIRTY_SECONDS /* TODO should this be configurable? longer by default? infinite? */, x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/WeightedAvg.java (1 line): - line 125: Object weightVal = weight.fold(FoldContext.small()/* TODO remove me*/); server/src/main/java/org/elasticsearch/common/lucene/Lucene.java (1 line): - line 574: // TODO: improve serialization of BigInteger x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/RemoteClusterSecurityLegacyCrossClusterApiKeysWithDlsFlsIT.java (1 line): - line 242: // TODO improve coverage to test: server/src/main/java/org/elasticsearch/common/time/JavaDateMathParser.java (1 line): - line 55: // TODO only millisecond granularity here! x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BytesRefLookup.java (1 line): - line 89: // TODO replace this with a warning and break x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/xpack/compute/operator/lookup/EnrichResultBuilderForBoolean.java (1 line): - line 84: // TODO: sort and dedup and set MvOrdering server/src/main/java/org/elasticsearch/index/mapper/BlockSourceReader.java (1 line): - line 54: ignoredValues.clear(); // TODO do something with these? x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/template/IndexTemplateRegistry.java (1 line): - line 83: private static final TimeValue REGISTRY_ACTION_TIMEOUT = TimeValue.THIRTY_SECONDS; // TODO should this be longer? server/src/main/java/org/elasticsearch/monitor/os/OsProbe.java (1 line): - line 791: // TODO: return a prettier name on non-Linux OS x-pack/plugin/enrich/src/internalClusterTest/java/org/elasticsearch/xpack/enrich/EnrichProcessorMaxMatchesIT.java (1 line): - line 47: // TODO Change this to run with security enabled x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/dataframe/process/AnalyticsResultProcessor.java (1 line): - line 110: // TODO When java 9 features can be used, we will not need the local variable here server/src/internalClusterTest/java/org/elasticsearch/gateway/RecoveryFromGatewayIT.java (1 line): - line 377: // TODO: remove once refresh doesn't fail immediately if there a master block: x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/job/persistence/ElasticsearchMappings.java (1 line): - line 212: // TODO make it the caller's responsibility to fork to an appropriate thread before even calling this method - see #87911 x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/util/NumericUtils.java (1 line): - line 134: if (unsignedLongMultiplyHigh(ux, uy) != 0) { // TODO: replace with Math#unsignedMultiplyHigh() in JDK 18 when available x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/FloatArrayState.java (1 line): - line 107: valuesBuilder.appendFloat(0); // TODO can we just use null? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/xpack/compute/operator/lookup/EnrichResultBuilderForLong.java (1 line): - line 84: // TODO: sort and dedup and set MvOrdering server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/DateHistogramAggregator.java (1 line): - line 272: // TODO: Stop using null here server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/DesiredBalanceComputer.java (1 line): - line 323: // TODO: we could include more cases that would cause early publishing of desired balance in case of a long server/src/main/java/org/elasticsearch/SpecialPermission.java (1 line): - line 57: // TODO: if we really need we can break out name (e.g. "hack" or "scriptEngineService" or whatever). x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/index/fielddata/LabelPositionVisitor.java (1 line): - line 20: * TODO: We could instead choose the point closer to the centroid which improves unbalanced trees server/src/main/java/org/elasticsearch/index/codec/tsdb/ES87TSDBDocValuesProducer.java (1 line): - line 753: // TODO: should we write to disk the actual max level for this segment? server/src/main/java/org/elasticsearch/action/admin/indices/resolve/ResolveIndexAction.java (1 line): - line 565: // TODO This is a dirty hack around the IndexNameExpressionResolver optimisation for "*" as described in: x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/saml/SamlMetadataCommand.java (1 line): - line 437: * @TODO REALM-SETTINGS[TIM] This can be redone a lot now the realm settings are keyed by type server/src/main/java/org/elasticsearch/common/geo/GeometryNormalizer.java (1 line): - line 247: // TODO: what happen with rectangles over the dateline x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/CombineDisjunctions.java (1 line): - line 144: // TODO: this makes a QL `or`, not an ESQL `or` modules/lang-expression/src/internalClusterTest/java/org/elasticsearch/script/expression/MoreExpressionIT.java (1 line): - line 55: // TODO: please convert to unit tests! x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/CcrSettings.java (1 line): - line 49: * TODO: Deprecate and remove this setting x-pack/plugin/core/src/main/plugin-metadata/entitlement-policy.yaml (1 line): - line 20: - set_https_connection_properties # TODO: review if we need this once we have proper test coverage server/src/main/java/org/elasticsearch/action/support/single/shard/TransportSingleShardAction.java (1 line): - line 78: // TODO: consider passing the executor, remove it from doExecute and let InboundHandler/TransportAction handle concurrency. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/termsenum/action/NodeTermsEnumRequest.java (1 line): - line 94: // TODO - if already timed out can we shortcut the trip somehow? Throw exception if remaining time < 0? plugins/discovery-ec2/src/javaRestTest/java/org/elasticsearch/discovery/ec2/DiscoveryEc2ClusterFormationTestCase.java (1 line): - line 57: // TODO also add some nodes to the DescribeInstances output which are filtered out, and verify that we do not x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/BooleanFunctionEqualsElimination.java (1 line): - line 36: // TODO: Replace use of QL Not with ESQL Not x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/X-BigArrayVector.java.st (1 line): - line 202: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/index/query/functionscore/DecayFunctionBuilder.java (1 line): - line 218: // TODO these ain't gonna work with runtime fields server/src/main/java/org/elasticsearch/action/search/TransportSearchScrollAction.java (1 line): - line 104: case QUERY_AND_FETCH_TYPE -> // TODO can we get rid of this? server/src/main/java/org/elasticsearch/indices/breaker/HierarchyCircuitBreakerService.java (1 line): - line 759: // TODO: use a threshold? Relative to % of memory? server/src/main/java/org/elasticsearch/index/reindex/AbstractBulkByScrollRequest.java (1 line): - line 414: // TODO It'd be nice not to refresh on every slice. Instead we should refresh after the sub requests finish. server/src/main/java/org/elasticsearch/common/lucene/search/function/FunctionScoreQuery.java (1 line): - line 518: return Float.MAX_VALUE; // TODO: what would be a good upper bound? server/src/main/java/org/elasticsearch/threadpool/DefaultBuiltInExecutorBuilders.java (1 line): - line 110: // TODO: remove (or refine) this temporary stateless custom refresh pool sizing once ES-7631 is solved. x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/IndexLifecycleRunner.java (1 line): - line 612: * TODO: refactor ILM logic so that this is not required any longer. It is unreasonably expensive to only filter out duplicate tasks at server/src/main/java/org/elasticsearch/search/aggregations/bucket/range/RangeAggregator.java (1 line): - line 431: // TODO correct the loss of precision from the range somehow.....? x-pack/plugin/monitoring/src/main/plugin-metadata/entitlement-policy.yaml (1 line): - line 5: # TODO: this is likely not needed, but including here to be on the safe side until server/src/main/java/org/elasticsearch/search/SearchPhaseResult.java (1 line): - line 129: // TODO: this seems wrong, SearchPhaseResult should have a writeTo? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/expression/function/scalar/string/BitLength.java (1 line): - line 42: // TODO investigate if a data type Long (BIGINT) wouldn't be more appropriate here modules/transport-netty4/src/internalClusterTest/java/org/elasticsearch/http/netty4/Netty4HttpRequestSizeLimitIT.java (1 line): - line 56: // TODO: We do not currently support in flight circuit breaker limits for bulk. However, IndexingPressure applies server/src/main/java/org/elasticsearch/index/mapper/DocValueFetcher.java (1 line): - line 28: // TODO rename this? It doesn't load from doc values, it loads from fielddata x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/xpack/compute/operator/lookup/EnrichResultBuilderForInt.java (1 line): - line 83: // TODO: sort and dedup and set MvOrdering x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/process/normalizer/ShortCircuitingRenormalizer.java (1 line): - line 168: if (latestAugmentedQuantiles != null) { // TODO: remove this if the assert doesn't trip in CI over the next year or so server/src/main/java/org/elasticsearch/search/aggregations/metrics/HyperLogLogPlusPlusSparse.java (1 line): - line 27: // TODO: consider a hll sparse structure x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/persistence/TransformInternalIndex.java (1 line): - line 393: TimeValue.THIRTY_SECONDS /* TODO should this be longer/configurable? */, x-pack/plugin/esql/qa/server/mixed-cluster/src/javaRestTest/java/org/elasticsearch/xpack/esql/qa/mixed/FieldExtractorIT.java (1 line): - line 40: // TODO these should all always return null because the parent is nested server/src/main/java/org/elasticsearch/index/fielddata/fieldcomparator/BytesRefFieldComparatorSource.java (1 line): - line 179: // TODO: move this out if we need it for other reasons x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/datafeed/extractor/aggregation/AggregationToJsonProcessor.java (1 line): - line 291: // TODO any validations or processing??? server/src/main/java/org/elasticsearch/common/settings/AbstractScopedSettings.java (1 line): - line 1009: // TODO this should be replaced by Setting.Property.HIDDEN or something like this. x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/BytesRefArrayState.java (1 line): - line 118: valuesBuilder.appendBytesRef(emptyBytesRef); // TODO can we just use null? x-pack/plugin/async-search/src/main/java/org/elasticsearch/xpack/search/TransportSubmitAsyncSearchAction.java (1 line): - line 112: // TODO: store intermediate results ? x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/action/TransportPutSnapshotLifecycleAction.java (1 line): - line 164: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/core/src/internalClusterTest/java/org/elasticsearch/xpack/core/termsenum/CCSTermsEnumIT.java (1 line): - line 35: // TODO Change this to run with security enabled x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/TransportPutAutoFollowPatternAction.java (1 line): - line 140: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/parser/LogicalPlanBuilder.java (1 line): - line 271: // TODO: unify this with the code from Join if the grammar gets aligned x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/Percentile.java (1 line): - line 172: return ((Number) percentile.fold(FoldContext.small() /* TODO remove me */)).intValue(); x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/enrich/EnrichLookupOperator.java (1 line): - line 165: // TODO: Maybe create a sub-task as the parent task of all the lookup tasks x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/SnapshotLifecycleTask.java (1 line): - line 189: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/search/aggregations/TimeSeriesGeoLineBuckets.java (1 line): - line 129: // TODO: For TSID perhaps we should save sortValues as longs x-pack/plugin/old-lucene-versions/src/main/java/org/elasticsearch/xpack/lucene/bwc/codecs/lucene70/fst/BytesStore.java (1 line): - line 31: // TODO: merge with PagedBytes, except PagedBytes doesn't server/src/main/java/org/elasticsearch/persistent/PersistentTasksCustomMetadata.java (1 line): - line 57: // TODO: Implement custom Diff for tasks x-pack/plugin/wildcard/src/main/java/org/elasticsearch/xpack/wildcard/mapper/BinaryDvConfirmedAutomatonQuery.java (1 line): - line 111: // TODO: how can we compute this? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/SpatialCentroid.java (1 line): - line 81: // TODO: Support geo_shape and cartesian_shape x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/categorization/CategorizeTextAggregator.java (1 line): - line 103: // TODO: make it possible to choose a language instead of or as well as English for the part-of-speech dictionary x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportMigrateToDataTiersAction.java (1 line): - line 185: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/DateHistogramValuesSourceBuilder.java (1 line): - line 226: // TODO once composite is plugged in to the values source registry or at least understands Date values source types use it server/src/main/java/org/elasticsearch/index/fielddata/IndexNumericFieldData.java (1 line): - line 114: // TODO: enable sort optimization for BYTE, SHORT and INT types server/src/internalClusterTest/java/org/elasticsearch/cluster/allocation/ClusterRerouteIT.java (1 line): - line 307: // TODO can we get around this? the cluster is RED, so what do we wait for? server/src/main/java/org/elasticsearch/index/codec/tsdb/es819/ES819TSDBDocValuesConsumer.java (1 line): - line 949: // TODO: can we do it in constant time rather than linearly with SKIP_INDEX_MAX_LEVEL? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/physical/EsQueryExec.java (1 line): - line 110: // TODO: refactor this: not all Sorts are backed by FieldAttributes x-pack/plugin/ent-search/src/main/java/org/elasticsearch/xpack/application/search/SearchApplicationIndexService.java (1 line): - line 89: * TODO: Revise the internal format (mappings). Should we use rest or transport versioning for BWC? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/type/EsqlDataTypeConverter.java (1 line): - line 211: // TODO move EXPRESSION_TO_LONG here if there is no regression x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/session/EsqlCCSUtils.java (1 line): - line 271: // TODO: this logic assumes a single phase execution model, so it may need to altered once INLINESTATS is made CCS compatible server/src/main/java/org/elasticsearch/index/shard/StoreRecovery.java (1 line): - line 564: // TODO no need to fork back to GENERIC if using cached repo data, see #101445 x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/completion/ElasticInferenceServiceCompletionModel.java (1 line): - line 124: // TODO create/refactor the Configuration class to be extensible for different task types (i.e completion, sparse embeddings). x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/FilteredExpression.java (1 line): - line 30: // TODO: This class should implement SurrogateExpression but it doesn't due to its use on folding aggregates x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/RemoteClusterSecurityDataStreamEsqlRcs1IT.java (1 line): - line 34: // TODO consolidate me with RemoteClusterSecurityDataStreamEsqlRcs2IT x-pack/plugin/monitoring/src/main/java/org/elasticsearch/xpack/monitoring/action/TransportMonitoringMigrateAlertsAction.java (1 line): - line 100: /* TODO expose separate ack timeout? use masterNodeTimeout() for now */ server/src/main/java/org/elasticsearch/index/fielddata/SortedBinaryDocValues.java (1 line): - line 21: // TODO: Should it expose a count (current approach) or return null when there are no more values? x-pack/plugin/core/src/main/java/org/elasticsearch/protocol/xpack/XPackInfoResponse.java (1 line): - line 36: // TODO move this constant to License.java once we move License.java to the protocol jar x-pack/plugin/ml-package-loader/src/main/java/org/elasticsearch/xpack/ml/packageloader/action/ModelLoaderUtils.java (1 line): - line 90: private final RequestRange range; // TODO debug only server/src/main/java/org/elasticsearch/injection/PlanInterpreter.java (1 line): - line 61: // TODO: switch patterns would make this unnecessary server/src/main/java/org/elasticsearch/cluster/metadata/MetadataCreateDataStreamService.java (1 line): - line 124: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanBigArrayVector.java (1 line): - line 132: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/injection/Planner.java (1 line): - line 101: // TODO: Better cycle detection and reporting. Use SCCs x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/service/ServiceAccountService.java (1 line): - line 177: // TODO: No production code usage x-pack/plugin/ml/src/internalClusterTest/java/org/elasticsearch/xpack/ml/integration/ChunkedTrainedModelRestorerIT.java (1 line): - line 174: // TODO is this the behaviour we want? server/src/main/java/org/elasticsearch/action/admin/cluster/node/tasks/cancel/TransportCancelTasksAction.java (1 line): - line 53: // implementations of CancellableTask#onCancelled() are nontrivial so we use GENERIC here. TODO could it be SAME? server/src/main/java/org/elasticsearch/search/aggregations/metrics/InternalTopHits.java (1 line): - line 109: // TODO: Can we do this better? x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/tree/Source.java (1 line): - line 75: // TODO: rename to location() x-pack/plugin/slm/src/main/java/org/elasticsearch/xpack/slm/action/TransportStopSLMAction.java (1 line): - line 62: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/profiling/src/main/java/org/elasticsearch/xpack/profiling/action/Resampler.java (1 line): - line 31: // TODO: Just use the sample rate as is once all resampling is done server-side x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/X-TopAggregator.java.st (1 line): - line 106: // TODO pass the breaker in from the DriverContext x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/user/User.java (1 line): - line 152: // TODO : reject `InternalUser` x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportJobTaskAction.java (1 line): - line 32: // TODO: Hacking around here with TransportTasksAction. Ideally we should have another base class in core that x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/lucene/TimeSeriesSourceOperator.java (1 line): - line 198: // TODO: add optimize for one leaf? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/Vector.java (1 line): - line 86: // TODO: Renaming this to owningBlockFactory modules/repository-s3/src/internalClusterTest/java/org/elasticsearch/repositories/s3/S3BlobStoreRepositoryTests.java (1 line): - line 741: // TODO track ListMultipartUploads requests x-pack/plugin/graph/src/main/java/org/elasticsearch/xpack/graph/action/TransportGraphExploreAction.java (1 line): - line 494: // TODO right now we only trim down to the best N vertices. We might also want to offer server/src/main/java/org/elasticsearch/common/lucene/search/function/ScriptScoreQuery.java (1 line): - line 295: return Float.MAX_VALUE; // TODO: what would be a good upper bound? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/EndsWith.java (1 line): - line 151: // TODO: Get the real FoldContext here x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/RemoteClusterSecurityRCS2ResolveClusterIT.java (1 line): - line 379: // TODO: fix this in a follow-on PR modules/lang-mustache/src/main/java/module-info.java (1 line): - line 17: exports org.elasticsearch.script.mustache; // TODO: remove once the HLRC is removed x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/execution/payload/EventPayload.java (1 line): - line 28: // TODO: remove unpooled usage x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/optimizer/OptimizerRules.java (1 line): - line 229: // TODO: eliminate conjunction/disjunction server/src/main/java/org/elasticsearch/cluster/RepositoryCleanupInProgress.java (1 line): - line 57: // TODO: Should we allow parallelism across repositories here maybe? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/ip/CIDRMatch.java (1 line): - line 192: Set set = new LinkedHashSet<>(Expressions.fold(FoldContext.small() /* TODO remove me */, matches)); x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/type/StringUtils.java (1 line): - line 27: //FIXME: this class comes from sql-proto server/src/main/java/org/elasticsearch/action/admin/cluster/health/TransportClusterHealthAction.java (1 line): - line 255: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/watcher/src/main/java/org/elasticsearch/xpack/watcher/notification/pagerduty/IncidentEvent.java (1 line): - line 148: // TODO externalize this into something user editable server/src/main/java/org/elasticsearch/indices/recovery/package-info.java (1 line): - line 184: * TODO: document other recovery types server/src/main/java/org/elasticsearch/index/query/TermsSetQueryBuilder.java (1 line): - line 379: // TODO: Change this to true when we can assume that scripts are pure functions x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/GroupingAggregatorFunction.java (1 line): - line 114: AddInput prepareProcessPage(SeenGroupIds seenGroupIds, Page page); // TODO allow returning null to opt out of the callback loop server/src/main/java/org/elasticsearch/index/seqno/ReplicationTracker.java (1 line): - line 289: // TODO improve this measure for when nested docs are in use x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/spatial/SpatialRelatesFunction.java (1 line): - line 216: Geometry shape = SpatialRelatesUtils.makeGeometryFromLiteral(FoldContext.small() /* TODO remove me */, constantExpression); x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/type/DataTypeConverter.java (1 line): - line 530: // TODO floating point conversions are lossy but conversions to integer are not. Are we ok with that? x-pack/plugin/vector-tile/src/main/java/org/elasticsearch/xpack/vectortile/rest/RestVectorTileAction.java (1 line): - line 136: // TODO: should we expose the total number of buckets on InternalGeoTileGrid? server/src/main/java/org/elasticsearch/action/search/SearchResponseMerger.java (1 line): - line 67: // TODO it may make sense to integrate the remote clusters responses as a shard response in the initial search phase and ignore hits coming x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/action/TransportDeleteTrainedModelAction.java (1 line): - line 248: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here modules/legacy-geo/src/main/java/org/elasticsearch/legacygeo/builders/EnvelopeBuilder.java (1 line): - line 78: // TODO support Z?? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/conditional/Greatest.java (1 line): - line 211: // TODO unsigned long server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/LongBounds.java (1 line): - line 152: // TODO: Should we rather pass roundUp=true? server/src/main/java/org/elasticsearch/index/IndexVersions.java (1 line): - line 51: // TODO: this is just a hack to allow to keep the V7 IndexVersion constants, during compilation. Remove x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/join/LookupJoin.java (1 line): - line 58: // TODO: decide whether to introduce USING or just basic ON semantics - keep the ordering out for now x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/TransportDeleteAutoFollowPatternAction.java (1 line): - line 71: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here libs/ssl-config/src/main/java/org/elasticsearch/common/ssl/SslConfiguration.java (1 line): - line 155: // TODO Add explicitlyConfigured to equals&hashCode? x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/process/MlMemoryTracker.java (1 line): - line 600: // TODO: does this also happen if the .ml-config index exists but is unavailable? server/src/internalClusterTest/java/org/elasticsearch/search/aggregations/bucket/NestedIT.java (1 line): - line 367: // TODO previously we would detect if you tried to do a nested agg on a non-nested object field, server/src/main/java/org/elasticsearch/script/BucketAggregationSelectorScript.java (1 line): - line 17: public abstract class BucketAggregationSelectorScript { // TODO move to the aggregation module x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/date/DateExtract.java (1 line): - line 156: // TODO: move the slimmed down code here to toEvaluator? server/src/main/java/org/elasticsearch/lucene/search/vectorhighlight/CustomFieldQuery.java (1 line): - line 35: // TODO: remove me! x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/registry/ModelRegistry.java (1 line): - line 965: // TODO: Add support to handle multi-projects x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/session/AbstractRowSet.java (1 line): - line 65: // TODO: this needs to be improved to properly scale each row across multiple lines server/src/main/java/org/elasticsearch/index/mapper/TimeSeriesIdFieldMapper.java (1 line): - line 120: // TODO don't leak the TSID's binary format into the script server/src/main/java/org/elasticsearch/cluster/ClusterFeatures.java (1 line): - line 125: // TODO: do we need some kind of transient cache of this calculation? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/spatial/SpatialExtentGroupingStateWrappedLongitudeState.java (1 line): - line 77: // TODO: Should we add Nulls here instead? x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/script/field/CartesianPointDocValuesField.java (1 line): - line 98: // TODO: merge common code with ScriptValues.GeoPoint server/src/main/java/org/elasticsearch/common/lucene/uid/PerThreadIDVersionAndSeqNoLookup.java (1 line): - line 50: // TODO: do we really need to store all this stuff? some if it might not speed up anything. server/src/main/java/org/elasticsearch/search/aggregations/support/ValueType.java (1 line): - line 119: // TODO: do not be lenient here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/parser/EsqlParser.java (1 line): - line 166: // TODO remove this at some point x-pack/license-tools/src/main/java/org/elasticsearch/license/licensor/tools/LicenseVerificationTool.java (1 line): - line 42: // TODO: with jopt-simple 5.0, we can make these requiredUnless each other modules/parent-join/src/internalClusterTest/java/org/elasticsearch/join/aggregations/AbstractParentChildTestCase.java (1 line): - line 57: // TODO: this array is always of length 1, and testChildrenAggs fails if this is changed server/src/main/java/org/elasticsearch/Version.java (1 line): - line 354: // TODO: 99 is leftover from alpha/beta/rc, it should be removed server/src/main/java/org/elasticsearch/inference/InferenceServiceRegistry.java (1 line): - line 33: // TODO check names are unique libs/entitlement/src/main/java/org/elasticsearch/entitlement/initialization/HardcodedEntitlements.java (1 line): - line 93: // TODO: what in es.base is accessing shared repo? x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/action/TransportGetCheckpointAction.java (1 line): - line 103: // TODO: do a quick check and only resolve if necessary?? modules/ingest-geoip/src/main/java/org/elasticsearch/ingest/geoip/EnterpriseGeoIpDownloader.java (1 line): - line 397: // TODO regardless of the above comment, i like the idea of checking the lowest last-checked time and then running the math to get server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/CompositeAggregator.java (1 line): - line 287: // TODO: can we handle missing bucket when using index sort optimization ? server/src/main/java/org/elasticsearch/search/lookup/SourceFilter.java (1 line): - line 56: // TODO: Remove this once we upgrade to Jackson 2.14. There is currently a bug x-pack/plugin/esql/qa/testFixtures/src/main/java/org/elasticsearch/xpack/esql/EsqlTestUtils.java (1 line): - line 847: // TODO degenerate versions and stuff server/src/main/java/org/elasticsearch/action/admin/indices/resolve/TransportResolveClusterAction.java (1 line): - line 94: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can server/src/internalClusterTest/java/org/elasticsearch/threadpool/SimpleThreadPoolIT.java (1 line): - line 107: if (threadName.contains("[node_s_0]") // TODO: this can't possibly be right! single node and integ test are unrelated! x-pack/plugin/ccr/src/yamlRestTest/java/org/elasticsearch/xpack/ccr/CcrRestIT.java (1 line): - line 27: // TODO: Switch to ingeg-test when we fix the xpack info api x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongArrayBlock.java (1 line): - line 134: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/math/Cast.java (1 line): - line 91: // TODO: catch-to-null in evaluator? x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/plugin/MatchOperatorIT.java (1 line): - line 91: // TODO: this should not raise an error; x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BytesRefArrayBlock.java (1 line): - line 143: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/search/vectors/KnnSearchBuilder.java (1 line): - line 63: // TODO optimize parsing for when BYTE values are provided server/src/internalClusterTest/java/org/elasticsearch/indices/store/IndicesStoreIntegrationIT.java (1 line): - line 367: // TODO: we should add an easier to do this. It's too much of a song and dance.. server/src/main/java/org/elasticsearch/common/settings/SecureSetting.java (1 line): - line 121: // TODO: override toXContent server/src/main/java/org/elasticsearch/lucene/search/uhighlight/BoundedBreakIteratorScanner.java (1 line): - line 33: * TODO: We should be able to create passages incrementally, starting from the offset of the first match and expanding or not server/src/main/java/org/elasticsearch/cluster/routing/allocation/command/CancelAllocationCommand.java (1 line): - line 211: // TODO: We don't have to remove a cancelled shard from in-sync set once we have a strict resync implementation. server/src/main/java/org/elasticsearch/cluster/project/ProjectResolver.java (1 line): - line 51: // TODO: What happens if the context does not have a project? throw or return null? server/src/main/java/org/elasticsearch/index/mapper/SourceLoader.java (1 line): - line 236: // TODO accept a requested xcontent type x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/BytesRef2BlockHash.java (1 line): - line 141: // TODO Build Ordinals blocks #114010 libs/entitlement/src/main/java/org/elasticsearch/entitlement/instrumentation/Transformer.java (1 line): - line 29: // TODO: Should warn if any MethodKey doesn't match any methods plugins/repository-hdfs/src/main/java/org/elasticsearch/repositories/hdfs/HdfsPlugin.java (1 line): - line 49: // TODO: remove THIS when hadoop is fixed x-pack/plugin/transform/src/internalClusterTest/java/org/elasticsearch/xpack/transform/integration/TransformInternalIndexIT.java (1 line): - line 43: // TODO Change this to run with security enabled server/src/main/java/org/elasticsearch/cluster/routing/allocation/allocator/BalancingRoundSummary.java (1 line): - line 88: * TODO: WIP ES-10341 plugins/discovery-ec2/src/main/java/org/elasticsearch/discovery/ec2/AwsEc2ServiceImpl.java (1 line): - line 41: // This is kinda pointless extra indirection; TODO fold it into Ec2DiscoveryPlugin modules/data-streams/src/main/java/org/elasticsearch/datastreams/action/TransportDeleteDataStreamAction.java (1 line): - line 128: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/TransportActivateAutoFollowPatternAction.java (1 line): - line 73: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/index/mapper/TextFieldMapper.java (1 line): - line 682: // TODO block loader could use a "fast loading" delegate which isn't always the same - but frequently is. server/src/main/java/org/elasticsearch/index/codec/tsdb/es819/XDocValuesConsumer.java (1 line): - line 837: // TODO: seek-by-ord to nextSetBit server/src/main/java/org/elasticsearch/common/settings/ConsistentSettingsService.java (1 line): - line 268: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/analytics/src/main/java/org/elasticsearch/xpack/analytics/aggregations/bucket/histogram/HistoBackedHistogramAggregator.java (1 line): - line 65: // TODO: Stop using null here x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/table/RowInTableLookup.java (1 line): - line 46: // TODO double check these errors over REST once we have LOOKUP server/src/main/java/org/elasticsearch/cluster/routing/allocation/command/MoveAllocationCommand.java (1 line): - line 172: // TODO we can possibly support also relocating cases, where we cancel relocation and move... x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/datafeed/DatafeedConfigUtils.java (1 line): - line 120: // TODO retains `dateHistogramInterval()`/`interval()` access for bwc logic, needs updating server/src/main/java/org/elasticsearch/action/explain/TransportExplainAction.java (1 line): - line 54: // TODO: AggregatedDfs. Currently the idf can be different then when executing a normal search with explain. server/src/main/java/org/elasticsearch/index/engine/CompletionStatsCache.java (1 line): - line 67: // TODO: currently we load up the suggester for reporting its size x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/transforms/TransformFailureHandler.java (1 line): - line 33: * TODO: x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/inference/trainedmodel/ensemble/Ensemble.java (1 line): - line 40: // TODO should we have regression/classification sub-classes that accept the builder? x-pack/plugin/security/cli/src/main/java/org/elasticsearch/xpack/security/cli/AutoConfigureNode.java (1 line): - line 717: // TODO add link to docs x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/tree/Source.java (1 line): - line 28: // TODO: rename to location() x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/nlp/FillMaskProcessor.java (1 line): - line 121: // TODO - process all results in the batch server/src/main/java/org/elasticsearch/search/SearchModule.java (1 line): - line 983: // TODO remove funky contexts x-pack/plugin/eql/qa/rest/src/javaRestTest/java/org/elasticsearch/xpack/eql/EqlRestValidationIT.java (1 line): - line 44: // TODO: revisit after x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/BooleanArrayState.java (1 line): - line 108: valuesBuilder.appendBoolean(false); // TODO can we just use null? server/src/main/java/org/elasticsearch/action/support/single/instance/InstanceShardOperationRequest.java (1 line): - line 25: // TODO: This request and its associated transport action can be folded into UpdateRequest which is its only concrete production code server/src/main/java/org/elasticsearch/transport/Transport.java (1 line): - line 260: // TODO: Only visible for testing. Perhaps move StubbableTransport from x-pack/plugin/sql/src/internalClusterTest/java/org/elasticsearch/xpack/sql/action/SqlLicenseIT.java (1 line): - line 156: // TODO test SqlGetIndicesAction. Skipping for now because of lack of serialization support. x-pack/plugin/esql/compute/gen/src/main/java/org/elasticsearch/compute/gen/EvaluatorImplementer.java (1 line): - line 281: // TODO: reflection on SingleValueQuery.MULTI_VALUE_WARNING? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/FloatFallibleArrayState.java (1 line): - line 109: valuesBuilder.appendFloat(0); // TODO can we just use null? modules/aggregations/src/main/java/org/elasticsearch/aggregations/metric/MatrixStatsAggregatorFactory.java (1 line): - line 59: // TODO: There must be a better option than this. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/action/GetDataFrameAnalyticsStatsAction.java (1 line): - line 257: // TODO: Have callers wrap the content with an object as they choose rather than forcing it upon them x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/action/BaseTransportInferenceAction.java (1 line): - line 149: // TODO: this is a temporary solution for passing around the product use case. server/src/main/java/org/elasticsearch/search/aggregations/bucket/histogram/DateRangeHistogramAggregator.java (1 line): - line 97: // TODO: Stop using null here server/src/main/java/org/elasticsearch/indices/analysis/wrappers/StableApiWrappers.java (1 line): - line 170: return org.elasticsearch.index.analysis.AnalyzerScope.GLOBAL;// TODO is this right? server/src/main/java/org/elasticsearch/action/support/master/MasterNodeRequest.java (1 line): - line 72: // TODO forbid TimeValue#MAX_VALUE once support for version prior to 8.14 dropped server/src/main/java/org/elasticsearch/index/reindex/ClientScrollableHitSource.java (1 line): - line 161: this.delegate = delegate.asUnpooled(); // TODO: use pooled version here server/src/main/java/org/elasticsearch/search/aggregations/bucket/composite/GeoTileGridValuesSourceBuilder.java (1 line): - line 86: // TODO: support adding circuit breaker in composite x-pack/plugin/sql/sql-cli/src/main/java/org/elasticsearch/xpack/sql/cli/Completers.java (1 line): - line 15: // TODO: need tree structure server/src/main/java/org/elasticsearch/common/io/stream/StreamOutput.java (1 line): - line 807: // TODO: improve serialization of BigInteger modules/transport-netty4/src/yamlRestTest/java/org/elasticsearch/http/netty4/Netty4ClientYamlTestSuiteIT.java (1 line): - line 24: //TODO: This is a *temporary* workaround to ensure a timeout does not mask other problems x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/action/TransportPutInferenceModelAction.java (1 line): - line 136: // required for BWC of elser service in elasticsearch service TODO remove when elser service deprecated x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/CommandLineHttpClient.java (1 line): - line 331: // TODO we still have passwords in Strings in headers. Maybe we can look into using a CharSequence? x-pack/plugin/ccr/src/main/java/org/elasticsearch/xpack/ccr/action/ShardFollowTasksExecutor.java (1 line): - line 570: // TODO: do we need minNodeVersion here since it is for remote cluster x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/security/authc/RealmSettings.java (1 line): - line 163: // TODO: Does it make sense to have empty realms for a domain? x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/util/DateUtils.java (1 line): - line 33: //FIXME: Taken from sql-proto (StringUtils) server/src/main/java/org/elasticsearch/index/codec/vectors/BQVectorUtils.java (1 line): - line 92: // TODO: move to VectorUtil & vectorize? benchmarks/src/main/java/org/elasticsearch/benchmark/compute/operator/BlockKeepMaskBenchmark.java (1 line): - line 106: // TODO float x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/TokenService.java (1 line): - line 2410: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/node/NodeService.java (1 line): - line 128: // TODO: revert to Build.current().version() when Kibana is updated x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/analysis/analyzer/Verifier.java (1 line): - line 679: // TODO: need to check whether it's possible to agg on a field used inside a scalar for grouping x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/exchange/ExchangeService.java (1 line): - line 59: // TODO: Make this a child action of the data node transport to ensure that exchanges x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/dataframe/analyses/Regression.java (1 line): - line 105: (regression) -> {/*TODO should we throw if this is not set?*/}, server/src/main/java/org/elasticsearch/search/rank/feature/RankFeatureDoc.java (1 line): - line 28: // TODO: update to support more than 1 fields; and not restrict to string data x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/io/stream/PlanStreamInput.java (1 line): - line 103: // TODO track blocks read over the wire.... Or slice them from BigArrays? Something. server/src/main/java/org/elasticsearch/search/aggregations/support/ValuesSource.java (1 line): - line 690: // TODO lookup the min and max rounding when appropriate x-pack/plugin/slm/src/javaRestTest/java/org/elasticsearch/xpack/slm/SnapshotLifecycleRestIT.java (1 line): - line 78: // TODO: Revert to integ-test distro once we sort out issues with usage and info xpack apis x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/DoubleArrayState.java (1 line): - line 107: valuesBuilder.appendDouble(0); // TODO can we just use null? modules/ingest-attachment/src/main/java/org/elasticsearch/ingest/attachment/AttachmentProcessor.java (1 line): - line 137: // TODO: stop using LanguageIdentifier... x-pack/plugin/migrate/src/main/java/org/elasticsearch/system_indices/action/TransportPostFeatureUpgradeAction.java (1 line): - line 95: TimeValue.THIRTY_SECONDS /* TODO should this be configurable? longer by default? infinite? */, x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/action/AbstractTransportSetResetModeAction.java (1 line): - line 115: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/core/src/main/java/org/elasticsearch/index/engine/frozen/FrozenEngine.java (1 line): - line 137: return indexCommit; // TODO maybe we can return an empty commit? x-pack/plugin/eql/qa/multi-cluster-with-security/src/javaRestTest/java/org/elasticsearch/xpack/eql/EqlRestValidationIT.java (1 line): - line 56: // TODO: revisit the next two tests when https://github.com/elastic/elasticsearch/issues/64190 is closed x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/notifications/InferenceAuditMessage.java (1 line): - line 20: // TODO this should be MODEL_ID... x-pack/plugin/searchable-snapshots/qa/azure/src/javaRestTest/java/org/elasticsearch/xpack/searchablesnapshots/AzureSearchableSnapshotsIT.java (1 line): - line 29: // TODO when https://github.com/elastic/elasticsearch/issues/111532 addressed, use a HTTPS fixture in FIPS mode too x-pack/plugin/searchable-snapshots/src/internalClusterTest/java/org/elasticsearch/xpack/searchablesnapshots/SearchableSnapshotsIntegTests.java (1 line): - line 119: // we can bypass this by forcing soft deletes to be used. TODO this restriction can be lifted when #55142 is resolved. x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/analysis/PostAnalyzer.java (1 line): - line 56: // TODO: this could be incorporated into the query generation x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/RateLongAggregator.java (1 line): - line 175: // TODO: add bulk_copy to Block x-pack/plugin/security/src/internalClusterTest/java/org/elasticsearch/xpack/security/profile/ProfileIntegTests.java (1 line): - line 650: // TODO: The index is created after the update call regardless. Should it not do that? x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/execution/search/extractor/MetricAggExtractor.java (1 line): - line 120: // TODO: need to investigate when this can be not-null x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/common/CartesianPoint.java (1 line): - line 71: // TODO add this method to SpatialPoint interface, allowing more code de-duplication x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/AbstractPageMappingToIteratorOperator.java (1 line): - line 320: // TODO a way to filter with a range please. plugins/discovery-gce/src/main/java/org/elasticsearch/discovery/gce/RetryHttpInitializerWrapper.java (1 line): - line 62: // TODO: figure out why GCE is so bad like this server/src/main/java/org/elasticsearch/cluster/routing/RoutingNode.java (1 line): - line 166: // TODO: change caller logic in RoutingNodes so that this check can go away x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/ConstantBooleanVector.java (1 line): - line 62: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/main/java/org/elasticsearch/search/suggest/phrase/LinearInterpolatingScorer.java (1 line): - line 18: //TODO public for tests x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/audit/AuditTrailService.java (1 line): - line 56: // TODO: this method only exists for access to LoggingAuditTrail in a Node for testing. x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/blockhash/CategorizePackedValuesBlockHash.java (1 line): - line 137: // TODO: This BytesStreamOutput is not accounted for by the circuit breaker. Fix that! x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlQueryTask.java (1 line): - line 47: // TODO it'd be nice to have the number of documents we've read from completed drivers here x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/fulltext/MultiMatch.java (1 line): - line 122: // TODO: add "zero_terms_query" x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/spatial/SpatialExtentGeoPointSourceValuesAggregator.java (1 line): - line 34: // TODO support non-longitude wrapped geo shapes. x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/search/aggregations/support/ShapeValuesSourceType.java (1 line): - line 20: // TODO (support scripts) server/src/main/java/org/elasticsearch/search/aggregations/support/AggregationInspectionHelper.java (1 line): - line 150: // TODO this could be incorrect... e.g. +1 + -1 x-pack/plugin/sql/jdbc/src/main/java/org/elasticsearch/xpack/sql/jdbc/JdbcConfiguration.java (1 line): - line 39: // TODO: beef this up for Security/SSL x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/TopBytesRefAggregator.java (1 line): - line 84: // TODO pass the breaker in from the DriverContext server/src/main/java/org/elasticsearch/persistent/PersistentTasksNodeService.java (1 line): - line 337: TimeValue.THIRTY_SECONDS /* TODO should this be longer? infinite? */, x-pack/plugin/searchable-snapshots/src/main/java/org/elasticsearch/xpack/searchablesnapshots/action/TransportMountSearchableSnapshotAction.java (1 line): - line 174: EsExecutors.DIRECT_EXECUTOR_SERVICE, // TODO fork to SNAPSHOT_META and drop the forking below, see #101445 x-pack/plugin/esql-core/src/main/java/org/elasticsearch/xpack/esql/core/util/SpatialCoordinateTypes.java (1 line): - line 101: // TODO: we should be able to transform WKT to WKB without building the geometry x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/BooleanArrayBlock.java (1 line): - line 155: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/esql/action/internal/SharedSecrets.java (1 line): - line 17: * TODO qualify export when ES|QL is modularized plugins/discovery-ec2/src/main/java/org/elasticsearch/discovery/ec2/AwsEc2Service.java (1 line): - line 25: // This is kinda pointless extra indirection and only has one implementation; TODO fold it into Ec2DiscoveryPlugin x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/categorization/CategorizeTextAggregatorFactory.java (1 line): - line 99: // TODO significant text does a 2x here, should we as well? x-pack/plugin/transform/src/main/java/org/elasticsearch/xpack/transform/TransformExtensionHolder.java (1 line): - line 17: * TODO: remove this class entirely once Guice is removed entirely. x-pack/plugin/security/qa/multi-cluster/src/javaRestTest/java/org/elasticsearch/xpack/remotecluster/RemoteClusterSecurityDataStreamEsqlRcs2IT.java (1 line): - line 26: // TODO consolidate me with RemoteClusterSecurityDataStreamEsqlRcs1IT modules/lang-painless/src/main/java/org/elasticsearch/painless/SimpleChecksAdapter.java (1 line): - line 25: // TODO: we should really try to get this fixed in ASM! server/src/main/java/org/elasticsearch/action/admin/indices/diskusage/IndexDiskUsageAnalyzer.java (1 line): - line 483: // TODO: Traverse 10-20% documents x-pack/plugin/esql/qa/server/multi-clusters/src/javaRestTest/java/org/elasticsearch/xpack/esql/ccq/MultiClusterSpecIT.java (1 line): - line 66: * TODO: Some spec tests prevents us from splitting data across multiple shards/indices/clusters x-pack/plugin/core/src/main/java/module-info.java (1 line): - line 69: exports org.elasticsearch.xpack.core.esql.action.internal; // TODO: qualify to esql when modularized x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/X-FallibleArrayState.java.st (1 line): - line 151: valuesBuilder.append$Type$($if(boolean)$false$else$0$endif$); // TODO can we just use null? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/data/BlockUtils.java (1 line): - line 228: // TODO: allow null values server/src/main/java/org/elasticsearch/health/node/selection/HealthNodeTaskExecutor.java (1 line): - line 157: TimeValue.THIRTY_SECONDS /* TODO should this be configurable? longer by default? infinite? */, x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/job/process/autodetect/NativeAutodetectProcess.java (1 line): - line 71: // TODO: should we fail to start? libs/tdigest/src/main/java/org/elasticsearch/tdigest/MergingDigest.java (1 line): - line 193: // TODO update with current numbers server/src/main/java/org/elasticsearch/index/get/GetResult.java (1 line): - line 246: // TODO: can we avoid having an exception here? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/QueryPlan.java (1 line): - line 180: // TODO if collection is a set then this silently changes its semantics by allowing duplicates server/src/internalClusterTest/java/org/elasticsearch/snapshots/SnapshotStressTestsIT.java (1 line): - line 1329: // TODO generalise this so that it succeeds as soon as it's acquired a permit on >1/2 of the master-eligible nodes x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/sagemaker/SageMakerClient.java (1 line): - line 193: // TODO migrate to entitlements server/src/main/java/org/elasticsearch/cluster/routing/BatchedRerouteService.java (1 line): - line 175: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/action/admin/cluster/reroute/TransportClusterRerouteAction.java (1 line): - line 209: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here x-pack/plugin/esql/qa/testFixtures/src/main/java/org/elasticsearch/xpack/esql/CsvTestUtils.java (1 line): - line 412: // TODO if we'd want escaped commas outside multi-values fields, we'd have to adjust this value here as well x-pack/plugin/ilm/src/main/java/org/elasticsearch/xpack/ilm/action/TransportStartILMAction.java (1 line): - line 58: @SuppressForbidden(reason = "legacy usage of unbatched task") // TODO add support for batching here server/src/main/java/org/elasticsearch/common/time/JavaDateFormatter.java (1 line): - line 62: // TODO ideally we should make defaulting for weekbased year here too, server/src/main/java/org/elasticsearch/indices/IndicesQueryCache.java (1 line): - line 98: // TODO avoid looping over all local shards here - see https://github.com/elastic/elasticsearch/issues/97222 server/src/main/java/org/elasticsearch/cluster/routing/GlobalRoutingTable.java (1 line): - line 125: * TODO: Remove this method, replace with routingTable(ProjectId) x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/xpack/compute/operator/lookup/EnrichResultBuilderForDouble.java (1 line): - line 84: // TODO: sort and dedup and set MvOrdering x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/aggs/categorization/TokenListSimilarityTester.java (1 line): - line 33: * TODO: It may be possible to apply some of the lesser optimisations x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/audit/AuditTrail.java (1 line): - line 100: * {@code BulkShardRequest}. We should strive to not use this and TODO refactor it out! x-pack/plugin/spatial/src/internalClusterTest/java/org/elasticsearch/xpack/spatial/search/CartesianShapeIntegTestCase.java (1 line): - line 32: // TODO: Support distance and bounding box queries on Cartesian data x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/transport/filter/IPFilter.java (1 line): - line 312: // FIXME we need to audit here server/src/main/java/org/elasticsearch/index/mapper/DocumentParserContext.java (1 line): - line 570: // TODO we may want to stop adding object mappers to the dynamic mappers list: most times they will be mapped when parsing their x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/ConstantFloatVector.java (1 line): - line 62: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. x-pack/plugin/mapper-aggregate-metric/src/main/java/org/elasticsearch/xpack/aggregatemetric/mapper/AggregateMetricDoubleFieldMapper.java (1 line): - line 364: return delegateFieldType().mayExistInIndex(context); // TODO how does searching actually work here? x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authc/saml/SamlRealm.java (1 line): - line 796: // @TODO : Submit a patch to OpenSAML to optionally disable the timer server/src/main/java/org/elasticsearch/action/admin/cluster/node/info/PluginsAndModules.java (1 line): - line 76: // TODO: not ideal, make a better api for this (e.g. with jar metadata, and so on) server/src/main/java/org/elasticsearch/plugins/IndexStorePlugin.java (1 line): - line 146: // TODO: remove unused API extension point build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/BaseInternalPluginBuildPlugin.java (1 line): - line 42: // TODO remove once we removed default dependencies from PluginBuildPlugin x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/LongLookup.java (1 line): - line 86: // TODO replace this with a warning and break x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/spatial/SpatialExtentGroupingState.java (1 line): - line 63: // TODO: Should we add Nulls here instead? x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/LongFallibleArrayState.java (1 line): - line 115: valuesBuilder.appendLong(0); // TODO can we just use null? server/src/main/java/org/elasticsearch/search/lookup/LeafFieldLookupProvider.java (1 line): - line 44: // TODO can we remember which fields have been loaded here and get them eagerly next time? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/DataNodeRequest.java (1 line): - line 85: // TODO make EsqlConfiguration Releasable x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/dataframe/steps/AnalysisStep.java (1 line): - line 66: // TODO This could fail with errors. In that case we get stuck with the copied index. x-pack/plugin/security/src/main/java/org/elasticsearch/xpack/security/authz/interceptor/BulkShardRequestInterceptor.java (1 line): - line 57: // TODO replace if condition with assertion x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plan/logical/join/InlineJoin.java (1 line): - line 67: * TODO: perform better planning server/src/main/java/org/elasticsearch/common/io/stream/RecyclerBytesStreamOutput.java (1 line): - line 160: // TODO: do this without copying the bytes from tmp by calling writeBytes and just use the pages in tmp directly through x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/xpack/compute/operator/lookup/EnrichResultBuilderForBytesRef.java (1 line): - line 101: // TODO: sort and dedup and set MvOrdering server/src/main/java/org/elasticsearch/index/mapper/SourceFieldMapper.java (1 line): - line 367: // nullable for bwc reasons - TODO: fold this into serializeMode x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/search/aggregations/metrics/CartesianShapeCentroidAggregator.java (1 line): - line 34: * TODO: This can be generalized with the nearly identical class GeoShapeCentroidAggregator x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/analysis/AnalyzerContext.java (1 line): - line 26: // TODO: make this even simpler, remove the enrichResolution for tests that do not require it (most tests) server/src/main/java/org/elasticsearch/cluster/metadata/MetadataUpdateSettingsService.java (1 line): - line 283: * TODO: should we update the in-sync allocation IDs once the data is deleted by the node? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/predicate/operator/comparison/InsensitiveEquals.java (1 line): - line 128: BytesRef value = BytesRefs.toBytesRef(valueOf(FoldContext.small() /* TODO remove me */, right())); modules/aggregations/src/main/java/org/elasticsearch/aggregations/metric/MatrixStatsAggregator.java (1 line): - line 100: // TODO: Fix matrix stats to treat neg inf as any other value x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/lookup/X-EnrichResultBuilder.java.st (1 line): - line 130: // TODO: sort and dedup and set MvOrdering x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/utils/time/DateTimeFormatterTimestampConverter.java (1 line): - line 91: @SuppressForbidden(reason = "TODO https://github.com/elastic/x-pack-elasticsearch/issues/3810") server/src/main/java/org/elasticsearch/gateway/TransportNodesListGatewayStartedShards.java (1 line): - line 135: // TODO: Fallback for BWC with older ES versions. Remove once request.getCustomDataPath() always returns non-null x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ml/search/SparseVectorQueryBuilder.java (1 line): - line 255: // TODO move this to xpack core and use inference APIs x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/ConstantIntVector.java (1 line): - line 62: // TODO if X-ArrayBlock used BooleanVector for it's null mask then we could shuffle references here. server/src/internalClusterTest/java/org/elasticsearch/action/admin/cluster/node/tasks/TasksIT.java (1 line): - line 356: // TODO: enhance this test to also check the tasks created by batched query execution x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/function/Function.java (1 line): - line 32: // TODO: Functions supporting distinct should add a dedicated constructor Location, List, boolean modules/aggregations/src/internalClusterTest/java/org/elasticsearch/aggregations/bucket/TimeSeriesAggregationsIT.java (1 line): - line 402: // TODO: Fix the top hits aggregation server/src/main/java/org/elasticsearch/action/support/ListenableActionFuture.java (1 line): - line 33: // The name {@link ListenableActionFuture} dates back a long way and could be improved - TODO find a better name x-pack/plugin/sql/qa/server/src/main/java/org/elasticsearch/xpack/sql/qa/jdbc/CsvSpecTestCase.java (1 line): - line 48: // TODO: https://github.com/elastic/elasticsearch/issues/40779 x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/spatial/SpatialExtentGeoPointDocValuesAggregator.java (1 line): - line 33: // TODO support non-longitude wrapped geo shapes. x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/execution/assembler/BoxedQueryRequest.java (1 line): - line 43: // TODO: performance testing to find the sweet spot server/src/main/java/org/elasticsearch/common/geo/SpatialPoint.java (1 line): - line 33: // TODO: Rather separate based on CRS, but since we don't have that yet, we use class name server/src/main/java/org/elasticsearch/action/bulk/BulkOperation.java (1 line): - line 618: // PRTODO: We could check for cluster feature here instead x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/RateDoubleAggregator.java (1 line): - line 175: // TODO: add bulk_copy to Block x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/multivalue/MvPSeriesWeightedSum.java (1 line): - line 99: // TODO figure out if the tests are wrong here, or if null is really different from foldable null x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/execution/search/BasicQueryClient.java (1 line): - line 169: // TODO: stop using unpooled x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/watcher/crypto/CryptoService.java (1 line): - line 50: // TODO: with better support in Java 8, we should consider moving to use AES GCM as it x-pack/plugin/ccr/src/internalClusterTest/java/org/elasticsearch/xpack/ccr/CcrDisabledIT.java (1 line): - line 21: // TODO: Assert that x-pack ccr feature is not enabled once feature functionality has been added x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/datafeed/persistence/DatafeedConfigProvider.java (1 line): - line 522: // TODO A better way to handle this rather than just ignoring the error? x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/aggregate/AggregateFunction.java (1 line): - line 98: && (filter.foldable() == false || Boolean.TRUE.equals(filter.fold(FoldContext.small() /* TODO remove me */)) == false); x-pack/plugin/spatial/src/main/java/org/elasticsearch/xpack/spatial/index/fielddata/GeoShapeValues.java (1 line): - line 100: * TODO: this is a test only method, perhaps should be moved to test code server/src/internalClusterTest/java/org/elasticsearch/action/admin/cluster/stats/ClusterStatsIT.java (1 line): - line 218: // TODO: Build.current().unqualifiedVersion() -- or Build.current().version() if/when we move NodeInfo to Build version(s) x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/StartsWith.java (1 line): - line 148: // TODO: Get the real FoldContext here x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/request/ElasticInferenceServiceAuthorizationRequest.java (1 line): - line 65: // TODO look into refactoring so we don't even need to return this, look at the RetryingHttpSender to fix this x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/fulltext/FullTextFunction.java (1 line): - line 126: Object queryAsObject = query().fold(FoldContext.small() /* TODO remove me */); x-pack/plugin/sql/sql-proto/src/main/java/org/elasticsearch/xpack/sql/proto/content/ParsedMediaType.java (1 line): - line 52: * TODO Do not rely on nulls x-pack/plugin/security/qa/security-basic/src/javaRestTest/java/org/elasticsearch/xpack/security/ManageRolesPrivilegeIT.java (1 line): - line 152: // TODO Should the new global role/manage privilege be listed here? Probably not because it's not documented x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/inference/loadingservice/ModelLoadingService.java (1 line): - line 455: // TODO Is there a way to only signal a cancel if all the listener tasks cancel??? server/src/main/java/org/elasticsearch/injection/guice/spi/InjectionPoint.java (1 line): - line 215: // TODO (crazybob): Filter out overridden members. x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/ChangePointOperator.java (1 line): - line 61: // TODO: make org.elasticsearch.xpack.esql.core.tree.Source available here server/src/main/java/org/elasticsearch/common/io/stream/DelayableWriteable.java (1 line): - line 125: // TODO: this path is currently not used in production code, if it ever is this should start using pooled buffers x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/scalar/string/Replace.java (1 line): - line 164: // TODO this is not right (inconsistent). See also https://github.com/elastic/elasticsearch/issues/100038 x-pack/plugin/eql/src/main/java/org/elasticsearch/xpack/eql/action/EqlSearchResponse.java (1 line): - line 311: // TODO: make this pooled? x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/IpArrayState.java (1 line): - line 133: valuesBuilder.appendBytesRef(scratch); // TODO can we just use null? x-pack/plugin/ql/src/main/java/org/elasticsearch/xpack/ql/expression/function/scalar/whitelist/InternalQlScriptUtils.java (1 line): - line 133: // TODO: this needs to be improved to avoid creating the pattern on every call server/src/main/java/org/elasticsearch/action/admin/cluster/node/info/NodesInfoRequestBuilder.java (1 line): - line 15: // TODO: This class's interface should match that of NodesInfoRequest x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/action/TransportDeleteInferenceEndpointAction.java (1 line): - line 81: // workaround for https://github.com/elastic/elasticsearch/issues/97916 - TODO remove this when we can x-pack/plugin/esql/qa/server/multi-clusters/src/javaRestTest/java/org/elasticsearch/xpack/esql/ccq/MultiClustersIT.java (1 line): - line 150: return false; // TODO: Version.CURRENT.onOrAfter(Version.V_8_13_0); ?? // the Async API was introduced in 8.13.0 x-pack/plugin/ml/src/main/java/org/elasticsearch/xpack/ml/MlMetrics.java (1 line): - line 438: // TODO: consider in the future, especially when we're at the stage of needing to upgrade serverless model snapshots x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/RateFloatAggregator.java (1 line): - line 176: // TODO: add bulk_copy to Block x-pack/plugin/sql/src/main/java/org/elasticsearch/xpack/sql/expression/function/scalar/string/UnaryStringFunction.java (1 line): - line 60: // TODO change this to use _source instead of the exact form (aka field.keyword for text fields) server/src/main/java/org/elasticsearch/script/CompositeFieldScript.java (1 line): - line 62: // TODO for now we re-run the script every time a leaf field is accessed, but we could cache the values? modules/lang-painless/src/main/java/org/elasticsearch/painless/phase/DefaultConstantFoldingOptimizationPhase.java (1 line): - line 719: // TODO find a better string to output server/src/main/java/org/elasticsearch/search/aggregations/metrics/AbstractInternalHDRPercentiles.java (1 line): - line 258: // TODO: upgrade the HDRHistogram library x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/lucene/LuceneTopNSourceOperator.java (1 line): - line 175: // TODO: share the bottom between shardCollectors modules/legacy-geo/src/main/java/org/elasticsearch/legacygeo/builders/ShapeBuilder.java (1 line): - line 64: // TODO how might we use JtsSpatialContextFactory to configure the context (esp. for non-geo)? server/src/main/java/org/elasticsearch/snapshots/InternalSnapshotsInfoService.java (1 line): - line 163: // TODO Maybe just clear out non-ongoing snapshot recoveries is the node is master eligible, so that we don't server/src/main/java/org/elasticsearch/rest/action/cat/RestIndicesAction.java (1 line): - line 589: // TODO: expose docs stats for replicated closed indices x-pack/plugin/downsample/src/main/java/org/elasticsearch/xpack/downsample/TransportDownsampleAction.java (1 line): - line 564: TimeValue.THIRTY_SECONDS /* TODO should this be configurable? longer by default? infinite? */, x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/topn/VersionTopNEncoder.java (1 line): - line 16: // TODO versions can contain nul so we need to delegate to the utf-8 encoder for the utf-8 parts of a version x-pack/plugin/esql/qa/server/single-node/src/javaRestTest/java/org/elasticsearch/xpack/esql/qa/single_node/PushQueriesIT.java (1 line): - line 249: // TODO figure out why this is sometimes here and sometimes not distribution/tools/server-cli/src/main/java/org/elasticsearch/server/cli/MachineDependentHeap.java (1 line): - line 55: // TODO: this could be more efficient, to only parse final options once x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/rules/logical/ReplaceAggregateNestedExpressionWithEval.java (1 line): - line 140: if (ex instanceof Attribute == false) { // TODO: foldables shouldn't require eval'ing either server/src/main/java/org/elasticsearch/action/admin/indices/create/TransportCreateIndexAction.java (1 line): - line 134: // TODO: This really needs the ID. But the current test depends on it going through the metadata to trigger more checks x-pack/plugin/core/src/main/java/org/elasticsearch/xpack/core/ilm/GenerateSnapshotNameStep.java (1 line): - line 138: // TODO: we are breaking the rules of UUIDs by lowercasing this here, find an alternative (snapshot names must be lowercase) x-pack/plugin/esql/src/internalClusterTest/java/org/elasticsearch/xpack/esql/action/LookupFromIndexIT.java (1 line): - line 142: // TODO lookup index mode doesn't seem to force a single shard. That'll break the lookup command. x-pack/plugin/async-search/src/internalClusterTest/java/org/elasticsearch/xpack/search/AsyncSearchErrorTraceIT.java (1 line): - line 56: // TODO: make this test work with batched query execution by enhancing ErrorTraceHelper.setupErrorTraceListener libs/x-content/src/main/java/org/elasticsearch/xcontent/XContentParserConfiguration.java (1 line): - line 59: // TODO: Remove when serverless uses the new API modules/repository-s3/src/main/plugin-metadata/entitlement-policy.yaml (1 line): - line 9: # TODO: check this is actually needed, and if we can isolate it to a test-only policy x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/data/IntLookup.java (1 line): - line 86: // TODO replace this with a warning and break server/src/main/java/org/elasticsearch/cluster/metadata/SystemIndexMetadataUpgradeService.java (1 line): - line 78: // TODO: we should have a more efficient way of getting just the changed indices so that we don't have to fork here server/src/main/java/org/elasticsearch/index/translog/BufferedChecksumStreamInput.java (1 line): - line 130: // TODO: support slicing here as well x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/operator/DriverContext.java (1 line): - line 80: // TODO maybe this should have a small fixed limit? server/src/main/java/org/elasticsearch/index/mapper/BlockLoader.java (1 line): - line 419: // TODO support non-singleton ords modules/analysis-common/src/main/java/org/elasticsearch/analysis/common/XUniqueTokenFilter.java (1 line): - line 26: // TODO remove this filter in 9.0.0 x-pack/plugin/esql/compute/src/main/generated-src/org/elasticsearch/compute/aggregation/DoubleFallibleArrayState.java (1 line): - line 109: valuesBuilder.appendDouble(0); // TODO can we just use null? server/src/main/java/org/elasticsearch/index/engine/ThreadPoolMergeScheduler.java (1 line): - line 460: // TODO is it possible that `estimatedMergeBytes` be `0` for correctly initialize merges, server/src/main/java/org/elasticsearch/cluster/ClusterChangedEvent.java (1 line): - line 140: // TODO: none of the usages of these `changedCustom` methods actually need the full list; they just want to know if a specific entry x-pack/plugin/sql/qa/jdbc/src/main/java/org/elasticsearch/xpack/sql/qa/jdbc/JdbcTestUtils.java (1 line): - line 61: * // TODO: add equality only once actually ported to 7.11 server/src/main/java/org/elasticsearch/index/mapper/FallbackSyntheticSourceBlockLoader.java (1 line): - line 106: // TODO figure out how to handle XContentDataHelper#voidValue() x-pack/plugin/enrich/src/internalClusterTest/java/org/elasticsearch/xpack/enrich/EnrichMultiNodeIT.java (1 line): - line 80: // TODO Change this to run with security enabled x-pack/plugin/esql/compute/src/main/java/org/elasticsearch/compute/aggregation/X-RateAggregator.java.st (1 line): - line 180: // TODO: add bulk_copy to Block x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/optimizer/LogicalPlanOptimizer.java (1 line): - line 186: // TODO: bifunction can now (since we now have just one data types set) be pushed into the rule modules/aggregations/src/internalClusterTest/java/org/elasticsearch/aggregations/pipeline/DateDerivativeIT.java (1 line): - line 88: // TODO: would be nice to have more random data here server/src/main/java/org/elasticsearch/search/aggregations/support/AggregationPath.java (1 line): - line 244: // TODO both unwrap and subAggregator are only used here! server/src/internalClusterTest/java/org/elasticsearch/index/store/CorruptedFileIT.java (1 line): - line 530: * TODO once checksum verification on snapshotting is implemented this test needs to be fixed or split into several modules/repository-azure/src/yamlRestTest/java/org/elasticsearch/repositories/azure/RepositoryAzureClientYamlTestSuiteIT.java (1 line): - line 36: // TODO when https://github.com/elastic/elasticsearch/issues/111532 addressed, use a HTTPS fixture in FIPS mode too