Skip to content

[MINOR] Fix the flag for bootstrap#10056

Merged
yihua merged 1 commit intoapache:masterfrom
linliu-code:fix_bootstrap_flag
Nov 10, 2023
Merged

[MINOR] Fix the flag for bootstrap#10056
yihua merged 1 commit intoapache:masterfrom
linliu-code:fix_bootstrap_flag

Conversation

@linliu-code
Copy link
Collaborator

Change Logs

Fix the flag.

Impact

Low.

Risk level (write none, low medium or high below)

Low.

  • The config description must be updated if new configs are added or the default value of the configs are changed
  • Any new feature or user-facing change requires updating the Hudi website. Please create a Jira ticket, attach the
    ticket number here and follow the instruction to make
    changes to the website.

Contributor's checklist

  • Read through contributor's guide
  • Change Logs and Impact were stated clearly
  • Adequate tests were added if applicable
  • CI passed

if (fileFormatUtils.isDefined) {
new HoodieMergeOnReadSnapshotHadoopFsRelationFactory(
sqlContext, metaClient, parameters, userSchema, isBootstrap = false).build()
sqlContext, metaClient, parameters, userSchema, isBootstrap = true).build()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch

@hudi-bot
Copy link
Collaborator

CI report:

Bot commands @hudi-bot supports the following commands:
  • @hudi-bot run azure re-run the last Azure build

@yihua yihua added priority:blocker Production down; release blocker release-1.0.0 labels Nov 10, 2023
@yihua
Copy link
Contributor

yihua commented Nov 10, 2023

Tests pass based on #10055.

@yihua yihua merged commit 83bb7ae into apache:master Nov 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:blocker Production down; release blocker release-1.0.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants