kennknowles commented on a change in pull request #13776:
URL: https://github.com/apache/beam/pull/13776#discussion_r560553176
##########
File path: website/www/site/content/en/documentation/dsls/sql/walkthrough.md
##########
@@ -139,8 +139,8 @@ to either a single `PCollection` or a `PCollectionTuple`
which holds multiple
// Create a PCollectionTuple containing both PCollections.
// TupleTags IDs will be used as table names in the SQL query
PCollectionTuple namesAndFoods = PCollectionTuple.of(
- new TupleTag<>("Apps"), appsRows), // appsRows from the previous
example
- new TupleTag<>("Reviews"), reviewsRows));
+ new TupleTag<>("Apps", appsRows), // appsRows from the previous example
Review comment:
Actually I think it should be
```
PCollectionTuple
.of(new TupleTag<>("Apps"), appsrows)
.and(new TupleTag<>("Reviews"), reviewsRows)
```
----------------------------------------------------------------
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:
[email protected]
