Skip to content

[Fix-14806][dolphinscheduler-dao] The syntax for changing this feature is not supported by PostgreSQL. #14807

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
GaoHu1997 wants to merge 2 commits into from
Closed

[Fix-14806][dolphinscheduler-dao] The syntax for changing this feature is not supported by PostgreSQL. #14807

GaoHu1997 wants to merge 2 commits into from

Conversation

@GaoHu1997
Copy link

@GaoHu1997 GaoHu1997 commented Aug 24, 2023 •
edited
Loading

image
postgresql does not support `

Closes #14806

@GaoHu1997 GaoHu1997 changed the title postgresql does not support , remove from the DqRuleInputEntryMapp… [Fix-14807][dolphinscheduler-dao] The syntax for changing this feature is not supported by PostgreSQL. Aug 24, 2023
@GaoHu1997 GaoHu1997 changed the title [Fix-14807][dolphinscheduler-dao] The syntax for changing this feature is not supported by PostgreSQL. [Fix-14806][dolphinscheduler-dao] The syntax for changing this feature is not supported by PostgreSQL. Aug 24, 2023
CallMeKingsley97
CallMeKingsley97 approved these changes Aug 25, 2023
View reviewed changes
Copy link
Contributor

@CallMeKingsley97 CallMeKingsley97 left a comment

Choose a reason for hiding this comment

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

LGTM

@qingwli
Copy link
Member

qingwli commented Sep 1, 2023

Related #14717

fuchanghai
fuchanghai approved these changes Sep 25, 2023
View reviewed changes
Copy link
Member

@fuchanghai fuchanghai left a comment

Choose a reason for hiding this comment

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

LGTM ,if pass CI

@qingwli qingwli mentioned this pull request Oct 8, 2023
Merged
@SbloodyS SbloodyS added the first time contributor First-time contributor label Oct 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@fuchanghai fuchanghai fuchanghai approved these changes

@caishunfeng caishunfeng Awaiting requested review from caishunfeng

@SbloodyS SbloodyS Awaiting requested review from SbloodyS SbloodyS is a code owner

+1 more reviewer

@CallMeKingsley97 CallMeKingsley97 CallMeKingsley97 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

backend first time contributor First-time contributor

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[sql syntax][dolphinscheduler-dao] DqRuleInputEntryMapper.xml exists postgresql does not support `

5 participants

@GaoHu1997 @qingwli @fuchanghai @CallMeKingsley97 @SbloodyS