fix: Fixed example materialize-incremental and improved explanation (#4734)

dandawg · web-flow · commit ca8a7ab888b5 · 2024-11-05T03:44:06.000Z
fixed example materialize-incremental and improved explanation

Signed-off-by: dandawg &lt;12484302+dandawg@users.noreply.github.com&gt;
diff --git a/docs/getting-started/quickstart.md b/docs/getting-started/quickstart.md
@@ -493,19 +493,14 @@ print(training_df.head())
 {% endtabs %}
 ### Step 6: Ingest batch features into your online store
 
-We now serialize the latest values of features since the beginning of time to prepare for serving (note: 
-`materialize-incremental` serializes all new features since the last `materialize` call).
+We now serialize the latest values of features since the beginning of time to prepare for serving. Note, `materialize_incremental` serializes all new features since the last `materialize` call, or since the time provided minus the `ttl` timedelta. In this case, this will be `CURRENT_TIME - 1 day` (`ttl` was set on the `FeatureView` instances in [feature_repo/feature_repo/example_repo.py](feature_repo/feature_repo/example_repo.py)).
 
 {% tabs %}
 {% tab title="Bash" %}
 ```bash
 CURRENT_TIME=$(date -u +"%Y-%m-%dT%H:%M:%S")
-# For mac
-LAST_YEAR=$(date -u -v -1y +"%Y-%m-%dT%H:%M:%S")
-# For Linux
-# LAST_YEAR=$(date -u -d "last year" +"%Y-%m-%dT%H:%M:%S")
 
-feast materialize-incremental $LAST_YEAR $CURRENT_TIME
+feast materialize-incremental $CURRENT_TIME
 ```
 {% endtab %}
 {% endtabs %}
diff --git a/examples/quickstart/quickstart.ipynb b/examples/quickstart/quickstart.ipynb
@@ -758,14 +758,14 @@
       "source": [
         "### Step 5a: Using `materialize_incremental`\n",
         "\n",
-        "We now serialize the latest values of features since the beginning of time to prepare for serving (note: `materialize_incremental` serializes all new features since the last `materialize` call).\n",
-        "\n",
-        "An alternative to using the CLI command is to use Python:\n",
+        "We now serialize the latest values of features since the beginning of time to prepare for serving. Note, `materialize_incremental` serializes all new features since the last `materialize` call, or since the time provided minus the `ttl` timedelta. In this case, this will be `CURRENT_TIME - 1 day` (`ttl` was set on the `FeatureView` instances in [feature_repo/feature_repo/example_repo.py](feature_repo/feature_repo/example_repo.py)). \n",
         "\n",
         "```bash\n",
         "CURRENT_TIME=$(date -u +\"%Y-%m-%dT%H:%M:%S\")\n",
         "feast materialize-incremental $CURRENT_TIME\n",
-        "```"
+        "```\n",
+        "\n",
+        "An alternative to using the CLI command is to use Python:"
       ]
     },
     {
@@ -1100,4 +1100,4 @@
   },
   "nbformat": 4,
   "nbformat_minor": 0
-}
+}
