Skip to content

Conversation

@flashJd
Copy link
Contributor

@flashJd flashJd commented Aug 18, 2022

Change Logs

Describe context and summary for this change. Highlight if any code was copied.

Impact

Describe any public API or user-facing feature change or any performance impact.

Risk level: none | low | medium | high

Choose one. If medium or high, explain what verification was done to mitigate the risks.

Contributor's checklist

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

@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:high Significant impact; potential bugs flink-sql labels Sep 6, 2022
@yihua
Copy link
Contributor

yihua commented Sep 6, 2022

@flashJd Similar here, could you explain what you're adding here?

@flashJd
Copy link
Contributor Author

flashJd commented Sep 6, 2022

@flashJd Similar here, could you explain what you're adding here?

now, flink-sql not support query -D data, so add a flink_config parameter to enable read -D data

@xushiyan xushiyan added the engine:flink Flink integration label Oct 31, 2022
@github-actions github-actions bot added the size:S PR with lines of changes in (10, 100] label Feb 26, 2024
@xushiyan xushiyan removed the flink-sql label Dec 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

engine:flink Flink integration priority:high Significant impact; potential bugs size:S PR with lines of changes in (10, 100]

Projects

Status: 🚧 Needs Repro
Status: 🔖 Ready for review

Development

Successfully merging this pull request may close these issues.

5 participants