Skip to content

[ci] only compile necessary modules for self hosted pipeline#2181

Merged
leonardBang merged 2 commits intoapache:masterfrom
whhe:pipeline-profile
Jun 13, 2023
Merged

[ci] only compile necessary modules for self hosted pipeline#2181
leonardBang merged 2 commits intoapache:masterfrom
whhe:pipeline-profile

Conversation

@whhe
Copy link
Copy Markdown
Member

@whhe whhe commented Jun 5, 2023

No description provided.

Comment thread pom.xml
Comment thread pom.xml

<profiles>
<profile>
<id>default</id>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The default and release are totally same ? If true do we need release profile?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The default and release are totally same ? If true do we need release profile?

The only difference is that there are some plugins for releasing in 'release' profile. I'm not familar about the releasing of Flink CDC, if we remove the release profile here, these configurations can still be set in local settings.xml.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Got it

Copy link
Copy Markdown
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

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

+1

Comment thread pom.xml

<profiles>
<profile>
<id>default</id>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Got it

@leonardBang leonardBang merged commit dbbef91 into apache:master Jun 13, 2023
spoon-lz pushed a commit to spoon-lz/flink-cdc-connectors that referenced this pull request Jun 13, 2023
whhe added a commit to whhe/flink-cdc-connectors that referenced this pull request Sep 19, 2023
ChaomingZhangCN pushed a commit to ChaomingZhangCN/flink-cdc that referenced this pull request Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants