codope commented on code in PR #8397:
URL: https://github.com/apache/hudi/pull/8397#discussion_r1186702216
##########
hudi-hadoop-mr/src/main/java/org/apache/hudi/hadoop/SchemaEvolutionContext.java:
##########
@@ -82,32 +81,25 @@ public class SchemaEvolutionContext {
private final InputSplit split;
private final JobConf job;
- private HoodieTableMetaClient metaClient;
+ private final HoodieTableMetaClient metaClient;
public Option<InternalSchema> internalSchemaOption;
- public SchemaEvolutionContext(InputSplit split, JobConf job) throws
IOException {
+ public SchemaEvolutionContext(InputSplit split, JobConf job) {
this(split, job, Option.empty());
}
- public SchemaEvolutionContext(InputSplit split, JobConf job,
Option<HoodieTableMetaClient> metaClientOption) throws IOException {
+ public SchemaEvolutionContext(InputSplit split, JobConf job,
Option<HoodieTableMetaClient> metaClientOption) {
this.split = split;
this.job = job;
this.metaClient = metaClientOption.isPresent() ? metaClientOption.get() :
setUpHoodieTableMetaClient();
Review Comment:
oh ok then there is no need of this PR.
--
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
