RustedBones commented on a change in pull request #16080:
URL: https://github.com/apache/beam/pull/16080#discussion_r764981595
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/coders/AvroCoder.java
##########
@@ -217,6 +217,7 @@ public static CoderProvider getCoderProvider() {
}
private final Class<T> type;
+ private final boolean useReflection;
Review comment:
I would say there is only one avro API and 3 implementations (Generic,
Specific & Reflect). This API is not specific to avro reflect so I removed the
suffix from the name. We can keep if if it creates issue.
--
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]
