Skip to content

Postgresql upsert bug#13874

Closed
epuaphios wants to merge 7 commits intoapache:devfrom
epuaphios:dev
Closed

Postgresql upsert bug#13874
epuaphios wants to merge 7 commits intoapache:devfrom
epuaphios:dev

Conversation

@epuaphios
Copy link

@epuaphios epuaphios commented Apr 5, 2023

Postgresql upsert bug

Screenshot from 2023-04-05 16-16-51

@github-actions github-actions bot added backend UI ui and front end related labels Apr 5, 2023
@epuaphios epuaphios closed this Apr 5, 2023
@github-actions github-actions bot removed UI ui and front end related backend labels Apr 5, 2023
@epuaphios epuaphios reopened this Apr 5, 2023
@epuaphios epuaphios changed the title presto driver update Postgresql upsert fix Apr 5, 2023
@epuaphios epuaphios changed the title Postgresql upsert fix Postgresql upsert bug Apr 5, 2023
@epuaphios epuaphios closed this Apr 5, 2023
@epuaphios epuaphios reopened this Apr 5, 2023
@davidzollo davidzollo added the first time contributor First-time contributor label Apr 9, 2023
INSERT INTO t_ds_trigger_relation (trigger_code, trigger_type, job_id, create_time, update_time) VALUES(
#{triggerRelation.triggerCode},#{triggerRelation.triggerType},#{triggerRelation.jobId},#{triggerRelation.createTime},#{triggerRelation.updateTime})
ON DUPLICATE KEY UPDATE update_time = #{triggerRelation.updateTime};
ON CONFLICT (trigger_code, trigger_type, job_id)
Copy link
Member

Choose a reason for hiding this comment

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

Could you please split this method to insert/update?

@epuaphios epuaphios requested a review from ruanwenjun April 10, 2023 19:38
Comment on lines +47 to +48
ports:
- "2181:2181"
Copy link
Member

Choose a reason for hiding this comment

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

export no public ports is not the best practice

Copy link
Author

Choose a reason for hiding this comment

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

yes but i running local test application not found zookeeper and postgresql

@SbloodyS SbloodyS added the 3.1.x for 3.1.x version label May 17, 2023
@SbloodyS SbloodyS added this to the 3.1.7 milestone May 17, 2023
@SbloodyS SbloodyS added the bug Something isn't working label May 17, 2023
@SbloodyS
Copy link
Member

Please resolve conflicts. @epuaphios

@zhuangchong zhuangchong modified the milestones: 3.1.7, 3.1.8 May 25, 2023
@zhuangchong zhuangchong modified the milestones: 3.1.8, 3.1.9 Aug 15, 2023
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

@github-actions github-actions bot added the Stale label Dec 15, 2023
@github-actions
Copy link

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.

@github-actions github-actions bot closed this Dec 23, 2023
@Alixxxxxxx
Copy link

3.2.2 this bug is also in.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3.1.x for 3.1.x version backend bug Something isn't working first time contributor First-time contributor Stale

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants