Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

[hotfix-#1497][chunjun-connector-http]Solve the problem that http-x p… #1500

Merged
merged 1 commit into from
Feb 13, 2023
Merged

[hotfix-#1497][chunjun-connector-http]Solve the problem that http-x p… #1500

merged 1 commit into from
Feb 13, 2023

Conversation

taoyameng
Copy link
Contributor

I Solve the problem that http-x plug-in cannot be used in sql mode
the issues address is
#1497

taoyameng
taoyameng commented Feb 13, 2023
View reviewed changes
Copy link
Contributor Author

@taoyameng taoyameng left a comment

Choose a reason for hiding this comment

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

that is ok

@FlechazoW
Copy link
Member

Thank you for your contribution. Before we merge your code, let's finish the ci/cd.

FlechazoW
FlechazoW approved these changes Feb 13, 2023
View reviewed changes
Copy link
Member

@FlechazoW FlechazoW left a comment

Choose a reason for hiding this comment

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

LGTM. Approve it.

@FlechazoW FlechazoW merged commit d65793b into DTStack:master Feb 13, 2023
@FlechazoW FlechazoW mentioned this pull request Feb 13, 2023
Closed
4 tasks
ll076110 pushed a commit that referenced this pull request Mar 3, 2023
@taoyameng
[hotfix-#1497][chunjun-connector-http]Solve the problem that http-x p…
986a026 
…lug-in cannot be used in sql mode (#1500)

Co-authored-by: 443321070@qq.com <taotao0226.?>
(cherry picked from commit d65793b)
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@FlechazoW FlechazoW FlechazoW approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@taoyameng @FlechazoW