Copilot commented on code in PR #37409:
URL: https://github.com/apache/beam/pull/37409#discussion_r2724374145
##########
examples/java/src/main/java/org/apache/beam/examples/cookbook/TriggerExample.java:
##########
@@ -486,7 +486,10 @@ static void runTriggerExample(TrafficFlowOptions options)
throws IOException {
.get(i)
.apply(
"BigQuery Write Rows" + i,
-
BigQueryIO.writeTableRows().to(tableRef).withSchema(getSchema()));
+ BigQueryIO.writeTableRows()
+ .to(tableRef)
+ .withSchema(getSchema())
+
.withWriteDisposition(BigQueryIO.Write.WriteDisposition.WRITE_APPEND));
Review Comment:
The indentation for the BigQueryIO builder chain is inconsistent with other
examples in the codebase. Most other examples (like AutoComplete.java:513-520,
CombinePerKeyExamples.java:197-201, WriteToBigQuery.java:129-134) use 4-space
indentation for chained methods, but this code uses a larger indentation (10
spaces). Consider adjusting the indentation to match the established pattern in
the codebase, aligning the chained methods at 4 spaces relative to the method
call context.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
