rezarokni commented on a change in pull request #12097: URL: https://github.com/apache/beam/pull/12097#discussion_r446765181
########## File path: examples/java/src/test/java/org/apache/beam/examples/snippets/SnippetsTest.java ########## @@ -154,6 +156,73 @@ public void testCoGroupByKeyTuple() throws IOException { p.run(); } + /* Tests SchemaJoinPattern */ + @Test + public void testSchemaJoinPattern() { + // [START SchemaJoinPatternCreate] + // Define Schemas + Schema emailSchema = + Schema.of( + Schema.Field.of("name", Schema.FieldType.STRING), + Schema.Field.of("email", Schema.FieldType.STRING)); + + Schema phoneSchema = + Schema.of( + Schema.Field.of("name", Schema.FieldType.STRING), + Schema.Field.of("phone", Schema.FieldType.STRING)); + + // Create User Data Collections + final List<Row> emailUsers = Review comment: Consider if adding a non-match would help the sample. For example a missing id from emailUsers or the other. So that the reader is clear on the output when no match occurs. ---------------------------------------------------------------- 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: us...@infra.apache.org