voonhous commented on code in PR #13927:
URL: https://github.com/apache/hudi/pull/13927#discussion_r2367005641


##########
hudi-common/src/main/java/org/apache/hudi/avro/HoodieAvroReaderContext.java:
##########
@@ -79,7 +80,24 @@ public HoodieAvroReaderContext(
       HoodieTableConfig tableConfig,
       Option<InstantRange> instantRangeOpt,
       Option<Predicate> filterOpt) {
-    this(storageConfiguration, tableConfig, instantRangeOpt, filterOpt, 
Collections.emptyMap());
+    this(storageConfiguration, tableConfig, instantRangeOpt, filterOpt, 
Collections.emptyMap(), tableConfig.getPayloadClass(), new HoodieConfig());

Review Comment:
   Added a `getReaderConfigs(storageConfiguration)` over here so that the 
reader configurations will be inferred whenever possible.
   
   However, the next question here is where storage configurations are read 
from, depending on code path, it could still be missing the Hfile 
configurations.



-- 
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]

Reply via email to