Skip to content
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

Add AbstractStatementExecutor #1269

Merged
merged 47 commits into from
Sep 17, 2018
Merged

Add AbstractStatementExecutor #1269

merged 47 commits into from
Sep 17, 2018

Conversation

tristaZero
Copy link
Contributor

Fixes #ISSUSE_ID.

Changes proposed in this pull request:

  • Add AbstractStatementExecutor.

@tristaZero
modify position
6419564
@tristaZero
initialize
cbb896f
@tristaZero
SQL exception
001ef70
@tristaZero
add AbstractStatementExecutor.java
867b078
@tristaZero
abstract function
8798da0
@tristaZero
delete private int batchCount;
ca9fb84
@tristaZero
use route
7c0856c
@tristaZero
delete this.returnGeneratedKeys = returnGeneratedKeys;
38afb1d
@tristaZero
add @Getter(AccessLevel.PROTECTED)
883b0b3
@tristaZero
move private SQLType sqlType;
218ed62
@tristaZero
MOVE private final Collection<RouteUnit> routeUnits = new LinkedList<…
ff9ec84 
…>();
@tristaZero
BLANK space
ff5620c
@tristaZero
add init()
754fa73
@tristaZero
getDatabaseType()
192bd32
@tristaZero
use getExecuteGroups()
b15ca89
@tristaZero
overwrite clear
7c8e851
@tristaZero
delete executeCallback()
16b981c
@tristaZero
MOVE @Getter
48aefcd
@tristaZero
extends AbstractStatementExecutor
2c50802
@tristaZero
modify init()
4bf9582
@tristaZero
use getDatabaseType()
f95132e
@tristaZero
delete abstract function
2ac5a21
@tristaZero
clear()
d26710f
@tristaZero
blank space
2348dee
@tristaZero
super(resultSetType, resultSetConcurrency, resultSetHoldability, sha…
aa41c82 
…rdingConnection);
@tristaZero
getResultSetConcurrency()
e4a197c
@tristaZero
resultSets.clear();
39bb633
@tristaZero
import java.util.concurrent.CopyOnWriteArrayList;
09ffb47
@tristaZero
executeGroups.clear();
d6a814c
@tristaZero
delete clear()
f816d0d
@coveralls
Copy link

Pull Request Test Coverage Report for Build 5096

  • 71 of 71 (100.0%) changed or added relevant lines in 5 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.1%) to 63.545%
Totals Coverage Status
Change from base Build 5095: -0.1%
Covered Lines: 8041
Relevant Lines: 12654
💛 - Coveralls

@terrymanu terrymanu merged commit d2d2598 into apache:dev Sep 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tristaZero @coveralls @terrymanu