Skip to content

Commit

Permalink
fix: referenced simple extension in tutorial (set instead of string) (#…
Browse files Browse the repository at this point in the history
…494)

I noticed that the tutorial has `functions_string.yaml` and
`functions_set.yaml`
mixed up for the `index_in` extension function. This PR fixes that.
  • Loading branch information
mbrobbel authored Apr 17, 2023
1 parent 5e99f0b commit b5d7ed2
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion site/docs/tutorial/final_plan.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
"extensionUris": [
{
"extensionUriAnchor": 1,
"uri": "https://github.com/substrait-io/substrait/blob/main/extensions/functions_string.yaml"
"uri": "https://github.com/substrait-io/substrait/blob/main/extensions/functions_set.yaml"
},
{
"extensionUriAnchor": 2,
Expand Down
4 changes: 2 additions & 2 deletions site/docs/tutorial/sql_to_substrait.md
Original file line number Diff line number Diff line change
Expand Up @@ -880,7 +880,7 @@ extension they come from.

In our query, we used:

* `index_in` (1), from `functions_string.yaml`,
* `index_in` (1), from `functions_set.yaml`,
* `is_null` (2), from `functions_comparison.yaml`,
* `equal` (3), from `functions_comparison.yaml`,
* `sum` (4), from `functions_arithmetic_decimal.yaml`,
Expand All @@ -892,7 +892,7 @@ So first we can create the three extension uris:
[
{
"extensionUriAnchor": 1,
"uri": "https://github.com/substrait-io/substrait/blob/main/extensions/functions_string.yaml"
"uri": "https://github.com/substrait-io/substrait/blob/main/extensions/functions_set.yaml"
},
{
"extensionUriAnchor": 2,
Expand Down

0 comments on commit b5d7ed2

Please sign in to comment.