alexeykudinkin commented on a change in pull request #4203:
URL: https://github.com/apache/hudi/pull/4203#discussion_r782641409
##########
File path: hudi-common/src/main/java/org/apache/hudi/avro/HoodieAvroUtils.java
##########
@@ -444,15 +445,15 @@ public static Schema generateProjectionSchema(Schema
originalSchema, List<String
/**
* Obtain value of the provided field as string, denoted by dot notation.
e.g: a.b.c
*/
- public static String getNestedFieldValAsString(GenericRecord record, String
fieldName, boolean returnNullIfNotFound) {
- Object obj = getNestedFieldVal(record, fieldName, returnNullIfNotFound);
+ public static String getNestedFieldValAsString(GenericRecord record, String
fieldName, boolean returnNullIfNotFound, boolean
consistentLogicalTimestampEnabled) {
Review comment:
@codope @nsivabalan folks, i don't believe this is the right fix to
address the problem: bubbling down this config value we're now exposing every
user of this API to be aware of it, which, very likely, will have very little
to do with it.
--
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]