-
Notifications
You must be signed in to change notification settings - Fork 976
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
Finetuning DbCp2 Parameters for Bulk Reader for string tall table connection drop issue. #1786
Finetuning DbCp2 Parameters for Bulk Reader for string tall table connection drop issue. #1786
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1786 +/- ##
============================================
+ Coverage 42.96% 49.75% +6.78%
+ Complexity 3398 1225 -2173
============================================
Files 816 353 -463
Lines 47687 19226 -28461
Branches 5123 1938 -3185
============================================
- Hits 20491 9566 -10925
+ Misses 25534 9008 -16526
+ Partials 1662 652 -1010
|
bd9f793
to
94e2adc
Compare
23fef84
to
b0e3620
Compare
...c/main/java/com/google/cloud/teleport/v2/source/reader/io/jdbc/iowrapper/JdbcDataSource.java
Show resolved
Hide resolved
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.
Minor comment - documentation link.
b0e3620
to
053d10c
Compare
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
Overview
While migrating a tall table (5TB+) with string primary keys, there exceptions of the type
java.io.EOFException: Can not read response from server. Expected to read 1,423 bytes, read 1,373 bytes before connection was unexpectedly lost.
causing the job to fail.After tuning the DBCP2 parameters to be more tuned towards long lived connections as needed by bulk migration , the exceptions are not seen in the tests.
TODO:
Exception