[GitHub] [beam] kennknowles commented on a change in pull request #12254: [BEAM-XXXX] Add note on zetaSQL dependencies

[GitBox <gi...@apache.org>]

kennknowles commented on a change in pull request #12254:
URL: https://github.com/apache/beam/pull/12254#discussion_r455302615



##########
File path: website/www/site/content/en/documentation/dsls/sql/overview.md
##########
@@ -55,8 +55,12 @@ For more information on the ZetaSQL features in Beam SQL, see the [Beam ZetaSQL
 
 To switch to Beam ZetaSQL, configure the [pipeline options](https://beam.apache.org/releases/javadoc/2.15.0/org/apache/beam/sdk/options/PipelineOptions.html) as follows:
 ```
-setPlannerName("org.apache.beam.sdk.extensions.sql.zetasql.ZetaSQLQueryPlanner")
+PipelineOptions options = ...;
+options
+    .as(BeamSqlPipelineOptions.class)
+    .setPlannerName("org.apache.beam.sdk.extensions.sql.zetasql.ZetaSQLQueryPlanner");
 ```
+Note, Use of the `ZetaSQLQueryPlanner` requires an additional dependency on `beam-sdks-java-extensions-sql-zetasql` in addition to the `beam-sdks-java-extensions-sql` package required for `CalcuiteQueryPlanner`.

Review comment:
       Typo: `CalciteQueryPlanner`




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org