[TRAFODION-3246] add slf4j in jdbc #1757
Conversation
|
Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/3057/ |
|
Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/3057/ |
|
@arvind-narain, @hegdean , @svarnau: Any comments on this one? |
|
This seems to be bundling in slf4j with JDBC driver. If that is the case, then the slf4j license, which is already in licenses/lic-server-bin needs to also be added to licenses/lic-clients-bin. |
|
thank you for your help Steve |
|
New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/3063/ |
|
Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/3063/ |
|
@svarnau, does this look good to you now? |
licenses/lic-clients-bin
Outdated
| @@ -32,3 +32,30 @@ MIT like license. (http://site.icu-project.org/) | |||
| this Software without prior written authorization of the copyright holder. | |||
|
|
|||
| =============================================================================== | |||
| The binary distribution of Apache Trafodion DCS and REST components bundle | |||
There was a problem hiding this comment.
here we should say JDBC driver instead of "DCS and REST components"
mashengchen
force-pushed
the
TRAFODION-3246
branch
from
1540df2
to
17c28c6
Compare
|
New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/3065/ |
|
Test Passed. https://jenkins.esgyn.com/job/Check-PR-master/3065/ |
|
is there some one can review this? |
|
@svarnau, does this look good to you now? |
add a new jar which can use slf4j to print log