[AMORO-3112]Use the same mysql transaction to handle multiple database operations#3114
[AMORO-3112]Use the same mysql transaction to handle multiple database operations#3114rfyu wants to merge 3 commits intoapache:masterfrom
Conversation
| TransactionFactory transactionFactory = new JdbcTransactionFactory(); | ||
| Environment environment = new Environment("develop", transactionFactory, dataSource); | ||
| Configuration configuration = new Configuration(environment); | ||
| configuration.setDefaultExecutorType(org.apache.ibatis.session.ExecutorType.BATCH); |
There was a problem hiding this comment.
AFAIK, this configuration only affects the statement object usage, it will not affect the transaction scope.
So I don't think we can solve the lock issue with this change.
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions. |
|
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
|
This pull request has been marked as stale due to 30 days of inactivity. It will be closed in 1 week if no further activity occurs. If you think that’s incorrect or this pull request requires a review, please simply write any comment. If closed, you can revive the PR at any time and @mention a reviewer or discuss it on the dev@amoro.apache.org list. Thank you for your contributions. |
|
This pull request has been closed due to lack of activity. This is not a judgement on the merit of the PR in any way. It is just a way of keeping the PR queue manageable. If you think that is incorrect, or the pull request requires review, you can revive the PR at any time. |
3 participants
Why are the changes needed?
Close #3112.
Brief change log
How was this patch tested?
Add some test cases that check the changes thoroughly including negative and positive cases if possible
Add screenshots for manual tests if appropriate
Run test locally before making a pull request
Documentation