nancyxu123 commented on code in PR #24390:
URL: https://github.com/apache/beam/pull/24390#discussion_r1035479345
##########
sdks/java/io/google-cloud-platform/src/main/java/org/apache/beam/sdk/io/gcp/spanner/changestreams/mapper/ChangeStreamRecordMapper.java:
##########
@@ -83,30 +90,41 @@ public class ChangeStreamRecordMapper {
private static final String CHILD_PARTITIONS_COLUMN = "child_partitions";
private static final String PARENT_PARTITION_TOKENS_COLUMN =
"parent_partition_tokens";
private static final String TOKEN_COLUMN = "token";
+ private final Dialect dialect;
+ private final JsonFormat.Printer printer;
+ private final JsonFormat.Parser parser;
- ChangeStreamRecordMapper() {}
+ ChangeStreamRecordMapper(Dialect dialect) {
+ this.dialect = dialect;
+
+ this.printer =
Review Comment:
I don't really see any documentation explicitly specifying this class as
thread-safe, unfortunately. However, like you said, it doesn't seem like these
objects will be accessed concurrently?
--
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]
