-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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.
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
refactor: executor class duplicate code #1448
refactor: executor class duplicate code #1448
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1448 +/- ##
=============================================
+ Coverage 45.43% 45.65% +0.21%
Complexity 1664 1664
=============================================
Files 345 345
Lines 12708 12641 -67
Branches 1611 1588 -23
=============================================
- Hits 5774 5771 -3
+ Misses 6287 6222 -65
- Partials 647 648 +1
Continue to review full report at Codecov.
|
break; | ||
} | ||
} catch (LockConflictException lce) { | ||
conn.rollback(sp); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should in while block?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should in while block?
done
fix import order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ⅰ. Describe what this PR did
remove duplicate code