[ES|QL] Take TOP_SNIPPETS out of snapshot

docs/changelog/139272.yaml

@@ -0,0 +1,5 @@
+pr: 139272
+summary: Take TOP_SNIPPETS out of snapshot
+area: ES|QL
+type: enhancement
+issues: []

docs/reference/query-languages/esql/_snippets/lists/string-functions.md

@@ -22,6 +22,7 @@
 * [`SPLIT`](../../functions-operators/string-functions.md#esql-split)
 * [`STARTS_WITH`](../../functions-operators/string-functions.md#esql-starts_with)
 * [`SUBSTRING`](../../functions-operators/string-functions.md#esql-substring)
+* [`TOP_SNIPPETS`](../../functions-operators/string-functions.md#esql-top_snippets)
 * [`TO_BASE64`](../../functions-operators/string-functions.md#esql-to_base64)
 * [`TO_LOWER`](../../functions-operators/string-functions.md#esql-to_lower)
 * [`TO_UPPER`](../../functions-operators/string-functions.md#esql-to_upper)

docs/reference/query-languages/esql/functions-operators/string-functions.md

@@ -87,6 +87,9 @@ mapped_pages:
 :::{include} ../_snippets/functions/layout/substring.md
 :::
 
+:::{include} ../_snippets/functions/layout/top_snippets.md
+:::
+
 :::{include} ../_snippets/functions/layout/to_base64.md
 :::
 

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/action/EsqlCapabilities.java

@@ -1732,7 +1732,7 @@ public enum Cap {
         /**
          * Returns the top snippets for given text content and associated query.
          */
-        TOP_SNIPPETS_FUNCTION(Build.current().isSnapshot()),
+        TOP_SNIPPETS_FUNCTION,
 
         // Last capability should still have a comma for fewer merge conflicts when adding new ones :)
         // This comment prevents the semicolon from being on the previous capability when Spotless formats the file.

x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/expression/function/EsqlFunctionRegistry.java

@@ -436,7 +436,8 @@ private static FunctionDefinition[][] functions() {
                 def(UrlEncode.class, UrlEncode::new, "url_encode"),
                 def(UrlEncodeComponent.class, UrlEncodeComponent::new, "url_encode_component"),
                 def(UrlDecode.class, UrlDecode::new, "url_decode"),
-                def(Chunk.class, bi(Chunk::new), "chunk") },
+                def(Chunk.class, bi(Chunk::new), "chunk"),
+                def(TopSnippets.class, tri(TopSnippets::new), "top_snippets") },
             // date
             new FunctionDefinition[] {
                 def(DateDiff.class, tric(DateDiff::new), "date_diff"),
@@ -574,8 +575,7 @@ private static FunctionDefinition[][] snapshotFunctions() {
                 def(L1Norm.class, L1Norm::new, "v_l1_norm"),
                 def(L2Norm.class, L2Norm::new, "v_l2_norm"),
                 def(Magnitude.class, Magnitude::new, "v_magnitude"),
-                def(Hamming.class, Hamming::new, "v_hamming"),
-                def(TopSnippets.class, tri(TopSnippets::new), "top_snippets") } };
+                def(Hamming.class, Hamming::new, "v_hamming") } };
     }
 
     public EsqlFunctionRegistry snapshotRegistry() {

x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/analysis/VerifierTests.java

@@ -3335,8 +3335,6 @@ public void testChunkFunctionInvalidInputs() {
     }
 
     public void testTopSnippetsFunctionInvalidInputs() {
-        assumeTrue("Requires top snippet function", EsqlCapabilities.Cap.TOP_SNIPPETS_FUNCTION.isEnabled());
-
         // Null field allowed
         query("from test | EVAL snippets = TOP_SNIPPETS(null, \"query\")");