leesf commented on a change in pull request #4193:
URL: https://github.com/apache/hudi/pull/4193#discussion_r763010421



##########
File path: hudi-common/src/main/java/org/apache/hudi/avro/HoodieAvroUtils.java
##########
@@ -454,6 +457,10 @@ public static String 
getNestedFieldValAsString(GenericRecord record, String fiel
    * Obtain value of the provided field, denoted by dot notation. e.g: a.b.c
    */
   public static Object getNestedFieldVal(GenericRecord record, String 
fieldName, boolean returnNullIfNotFound) {
+    if (StringUtils.isNullOrEmpty(fieldName.trim())) {

Review comment:
       why here the fieldName be empty if we use default field




-- 
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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to