vamshigv commented on code in PR #9539:
URL: https://github.com/apache/hudi/pull/9539#discussion_r1308102765
##########
hudi-common/src/main/java/org/apache/hudi/common/bootstrap/index/HFileBootstrapIndex.java:
##########
@@ -182,12 +182,8 @@ private static String getUserKeyFromCellKey(String
cellKey) {
* @param fileSystem File System
*/
private static HFile.Reader createReader(String hFilePath, Configuration
conf, FileSystem fileSystem) {
- try {
- LOG.info("Opening HFile for reading :" + hFilePath);
- return HoodieHFileUtils.createHFileReader(fileSystem, new
HFilePathForReader(hFilePath), new CacheConfig(conf), conf);
- } catch (IOException ioe) {
- throw new HoodieIOException(ioe.getMessage(), ioe);
- }
+ LOG.info("Opening HFile for reading :" + hFilePath);
Review Comment:
This is removing try-catch block, is this intentional ?
--
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]
