-
Notifications
You must be signed in to change notification settings - Fork 501
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
[INLONG-1670][Manager] Add H2 in UT #1671
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1671 +/- ##
============================================
- Coverage 12.21% 12.20% -0.02%
- Complexity 1048 1050 +2
============================================
Files 392 392
Lines 32755 32755
Branches 5159 5159
============================================
- Hits 4001 3997 -4
- Misses 27989 27993 +4
Partials 765 765
Continue to review full report at Codecov.
|
@healchow PTAL, thanks. |
@Test | ||
public void dataSource() { | ||
DataSource dataSource = inLongManagerDatasourceConfig.dataSource(); | ||
System.out.println(dataSource); |
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.
please remove the println code.
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.
Thanks, done.
3a72d38
to
96e0e98
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
close #1670