Skip to content

[HUDI-5568] incorrect use of fileSystemView#7685

Merged
danny0405 merged 15 commits intoapache:masterfrom
loukey-lj:HUDI-5568
Jan 31, 2023
Merged

[HUDI-5568] incorrect use of fileSystemView#7685
danny0405 merged 15 commits intoapache:masterfrom
loukey-lj:HUDI-5568

Conversation

@loukey-lj
Copy link
Copy Markdown
Contributor

Change Logs

writeClient.getHoodieTable().getFileSystemView() always return the local fileSystemView,

should use writeClient. getHoodieTable(). getHoodieView() to determine the fileSystemView

Impact

NA

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

NA

Documentation Update

NA

Contributor's checklist

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

@loukey-lj loukey-lj changed the title [HUDI 5568] [HUDI-5568] Jan 17, 2023
@loukey-lj loukey-lj changed the title [HUDI-5568] [HUDI-5568] incorrect use of fileSystemView Jan 17, 2023
@loukey-lj
Copy link
Copy Markdown
Contributor Author

hi @danny0405 could you please take a look

@hudi-bot
Copy link
Copy Markdown
Collaborator

CI report:

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

@danny0405 danny0405 added engine:flink Flink integration writer-core priority:critical Production degraded; pipelines stalled priority:blocker Production down; release blocker and removed priority:critical Production degraded; pipelines stalled labels Jan 31, 2023
@danny0405 danny0405 self-assigned this Jan 31, 2023
@danny0405 danny0405 merged commit d90f286 into apache:master Jan 31, 2023
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Jan 31, 2023
…esystem instance instead (apache#7685)

Should use `writeClient. getHoodieTable(). getHoodieView()` to determine the fileSystemView
yihua pushed a commit that referenced this pull request Jan 31, 2023
…esystem instance instead (#7685)

Should use `writeClient. getHoodieTable(). getHoodieView()` to determine the fileSystemView
nsivabalan pushed a commit to nsivabalan/hudi that referenced this pull request Mar 22, 2023
…esystem instance instead (apache#7685)

Should use `writeClient. getHoodieTable(). getHoodieView()` to determine the fileSystemView
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Apr 5, 2023
…esystem instance instead (apache#7685)

Should use `writeClient. getHoodieTable(). getHoodieView()` to determine the fileSystemView
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine:flink Flink integration priority:blocker Production down; release blocker

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants