Fix put rollup job api documentation (#3637)

* Fix put rollup job api documentation Signed-off-by: Tanqiu Liu <[email protected]> * Apply suggestions from code review Co-authored-by: Melissa Vagi <[email protected]> --------- Signed-off-by: Tanqiu Liu <[email protected]> Co-authored-by: Aria Marble <[email protected]> Co-authored-by: Melissa Vagi <[email protected]> (cherry picked from commit 165e4cb) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
opensearch-project · Apr 10, 2023 · 9cb5f91 · 9cb5f91
1 parent 47fd39c
commit 9cb5f91
Showing 1 changed file with 102 additions and 50 deletions.
diff --git a/_im-plugin/index-rollups/rollup-api.md b/_im-plugin/index-rollups/rollup-api.md
@@ -50,38 +50,42 @@ PUT _plugins/_rollup/jobs/<rollup_id>?if_seq_no=1&if_primary_term=1 // Update
       "example_rollup_index_all"
     ],
     "continuous": false,
-    "dimensions": {
-      "date_histogram": {
-        "source_field": "tpep_pickup_datetime",
-        "fixed_interval": "1h",
-        "timezone": "America/Los_Angeles"
-      },
-      "terms": {
-        "source_field": "PULocationID"
+    "dimensions": [
+      {
+        "date_histogram": {
+          "source_field": "tpep_pickup_datetime",
+          "fixed_interval": "1h",
+          "timezone": "America/Los_Angeles"
+        }
       },
-      "metrics": [
-        {
-          "source_field": "passenger_count",
-          "metrics": [
-            {
-              "avg": {}
-            },
-            {
-              "sum": {}
-            },
-            {
-              "max": {}
-            },
-            {
-              "min": {}
-            },
-            {
-              "value_count": {}
-            }
-          ]
+      {
+        "terms": {
+          "source_field": "PULocationID"
         }
-      ]
-    }
+      }
+    ],
+    "metrics": [
+      {
+        "source_field": "passenger_count",
+        "metrics": [
+          {
+            "avg": {}
+          },
+          {
+            "sum": {}
+          },
+          {
+            "max": {}
+          },
+          {
+            "min": {}
+          },
+          {
+            "value_count": {}
+          }
+        ]
+      }
+    ]
   }
 }
 ```
@@ -103,33 +107,81 @@ Options | Description | Type | Required
 `description` | Optionally, describe the rollup job. | String | No
 `enabled` | When true, the index rollup job is scheduled. Default is true. | Boolean | Yes
 `continuous` | Specify whether or not the index rollup job continuously rolls up data forever or just executes over the current data set once and stops. Default is false. | Boolean | Yes
-`error_notification` | Set up a Mustache message template sent for error notifications. For example, if an index rollup job fails, the system sends a message to a Slack channel. | Object | No
-`page_size` | Specify the number of buckets to paginate through at a time while rolling up. | Number | Yes
+`error_notification` | Set up a Mustache message template for error notifications. For example, if an index rollup job fails, the system sends a message to a Slack channel. | Object | No
+`page_size` | Specify the number of buckets to paginate at a time during rollup. | Number | Yes
 `delay` | The number of milliseconds to delay execution of the index rollup job. | Long | No
-`dimensions` | Specify aggregations to create dimensions for the roll up time window. | Object | Yes
-`dimensions.date_histogram` | Specify either fixed_interval or calendar_interval, but not both. Either one limits what you can query in the target index. | Object | No
-`dimensions.date_histogram.fixed_interval` | Specify the fixed interval for aggregations in milliseconds, seconds, minutes, hours, or days. | String | No
-`dimensions.date_histogram.calendar_interval` | Specify the calendar interval for aggregations in minutes, hours, days, weeks, months, quarters, or years. | String | No
-`dimensions.date_histogram.field` | Specify the date field used in date histogram aggregation. | String | No
-`dimensions.date_histogram.timezone` | Specify the timezones as defined by the IANA Time Zone Database. The default is UTC. | String | No
-`dimensions.terms` | Specify the term aggregations that you want to roll up. | Object | No
-`dimensions.terms.fields` | Specify terms aggregation for compatible fields. | Object | No
-`dimensions.histogram` | Specify the histogram aggregations that you want to roll up. | Object | No
-`dimensions.histogram.field` | Add a field for histogram aggregations. | String | Yes
-`dimensions.histogram.interval` | Specify the histogram aggregation interval for the field. | Long | Yes
-`dimensions.metrics` | Specify a list of objects that represent the fields and metrics that you want to calculate. | Nested object | No
-`dimensions.metrics.field` | Specify the field that you want to perform metric aggregations on. | String | No
-`dimensions.metrics.field.metrics` | Specify the metric aggregations you want to calculate for the field. | Multiple strings | No
+`dimensions` | Specify aggregations to create dimensions for the roll up time window. Supported groups are `terms`, `histogram`, and `date_histogram`. For more information, see [Bucket Aggregations]({{site.url}}{{site.baseurl}}/opensearch/bucket-agg). | Array | Yes
+`metrics` | Specify a list of objects that represent the fields and metrics that you want to calculate. Supported metrics are `sum`, `max`, `min`, `value_count` and `avg`. For more information, see [Metric Aggregations]({{site.url}}{{site.baseurl}}/opensearch/metric-agg). | Array | No
 
 
 #### Example response
 
 ```json
 {
-  "_id": "rollup_id",
-  "_seqNo": 1,
-  "_primaryTerm": 1,
-  "rollup": { ... }
+  "_id": "<rollup_id>",
+  "_version": 3,
+  "_seq_no": 1,
+  "_primary_term": 1,
+  "rollup": {
+    "rollup_id": "<rollup_id>",
+    "enabled": true,
+    "schedule": {
+      "interval": {
+        "start_time": 1680159934649,
+        "period": 1,
+        "unit": "Days",
+        "schedule_delay": 0
+      }
+    },
+    "last_updated_time": 1680159934649,
+    "enabled_time": 1680159934649,
+    "description": "Example rollup job",
+    "schema_version": 17,
+    "source_index": "nyc-taxi-data",
+    "target_index": "rollup-nyc-taxi-data",
+    "metadata_id": null,
+    "page_size": 200,
+    "delay": 0,
+    "continuous": false,
+    "dimensions": [
+      {
+        "date_histogram": {
+          "fixed_interval": "1h",
+          "source_field": "tpep_pickup_datetime",
+          "target_field": "tpep_pickup_datetime",
+          "timezone": "America/Los_Angeles"
+        }
+      },
+      {
+        "terms": {
+          "source_field": "PULocationID",
+          "target_field": "PULocationID"
+        }
+      }
+    ],
+    "metrics": [
+      {
+        "source_field": "passenger_count",
+        "metrics": [
+          {
+            "avg": {}
+          },
+          {
+            "sum": {}
+          },
+          {
+            "max": {}
+          },
+          {
+            "min": {}
+          },
+          {
+            "value_count": {}
+          }
+        ]
+      }
+    ]
+  }
 }
 ```