Skip to content

[HUDI-5404] Add flink bundle validation#7500

Merged
xushiyan merged 4 commits intoapache:masterfrom
xushiyan:HUDI-5404-flink-bundle-validation
Dec 19, 2022
Merged

[HUDI-5404] Add flink bundle validation#7500
xushiyan merged 4 commits intoapache:masterfrom
xushiyan:HUDI-5404-flink-bundle-validation

Conversation

@xushiyan
Copy link
Member

Change Logs

Add bundle validation flow for flink bundles in GH actions CI.

Impact

NA

Risk level

None.

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

@xushiyan xushiyan added priority:high Significant impact; potential bugs engine:flink Flink integration labels Dec 18, 2022
@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

@xushiyan xushiyan requested a review from danny0405 December 19, 2022 03:58
RUN if [[ $SPARK_HOME == *"spark-3.2"* ]] || [[ $SPARK_HOME == *"spark-3.3"* ]]; \
then printf "\nspark.sql.catalog.spark_catalog org.apache.spark.sql.hudi.catalog.HoodieCatalog\n" >> $SPARK_HOME/conf/spark-defaults.conf; fi
RUN printf "\ntaskmanager.numberOfTaskSlots: 2\n" >> $FLINK_HOME/conf/flink-conf.yaml
RUN printf "\nlocalhost\n" >> $FLINK_HOME/conf/workers
Copy link
Contributor

Choose a reason for hiding this comment

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

It means the validation only has 1 TM(task manager) with 2 slots, which should be okey for the test because the compaction task parallelisim expects to be 1, just to note that this could be risky that we may run out of slots.

Copy link
Contributor

@danny0405 danny0405 left a comment

Choose a reason for hiding this comment

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

+1

@xushiyan xushiyan merged commit a0fb9b2 into apache:master Dec 19, 2022
@xushiyan xushiyan deleted the HUDI-5404-flink-bundle-validation branch December 19, 2022 07:37
fengjian428 pushed a commit to fengjian428/hudi that referenced this pull request Apr 5, 2023
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

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

4 participants