ktblsva commented on PR #12774: URL: https://github.com/apache/hudi/pull/12774#issuecomment-2642520597
> > Remove param which is not relevant to non-hadoop fs > > Are you saying the API for non-hadoop fs is not simple enough? this is what was written in the task: > HoodieStorage.openSeekable is needed to wrap the input stream when reading log files in Hadoop. We had to specify this boolean value in the API, but should not do so as it will not be relevant to other file systems. so yes, looks like wrapStream param needs only for read log files in hadoop and nowhere else -- 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]
