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

Print the java test logs for YSQL Connection Manager Unit Tests #22531

Closed
yugabyte-ci opened this issue May 24, 2024 · 0 comments
Closed

Print the java test logs for YSQL Connection Manager Unit Tests #22531

yugabyte-ci opened this issue May 24, 2024 · 0 comments
Assignees
@vpatibandla-yb
Labels
area/ysql Yugabyte SQL (YSQL) jira-originated kind/new-feature This is a request for a completely new feature priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage

Comments

@yugabyte-ci
Copy link
Contributor

yugabyte-ci commented May 24, 2024 •
edited by jira bot
Loading

Jira Link: DB-11457
@yugabyte-ci yugabyte-ci added jira-originated kind/new-feature This is a request for a completely new feature priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage labels May 24, 2024
@vpatibandla-yb vpatibandla-yb changed the title Print the java test logs as part of unit tests Print the java test logs for YSQL Connection Manager Unit Tests May 30, 2024
@yugabyte-ci yugabyte-ci added the area/ysql Yugabyte SQL (YSQL) label May 31, 2024
vpatibandla-yb added a commit that referenced this issue May 31, 2024
@vpatibandla-yb
[#22531] YSQL: Print the java test logs for YSQL Connection Manager U…
944b9aa 
…nit Tests

Summary:
The java test logs are not getting logged except for the ERROR messages. This is because the test class path contains multiple SLF4J bindings. The below message was noticed in the beginning of the logs.

```
SLF4J: Class path contains multiple SLF4J bindings.
SLF4J: Found binding in [jar:file:~/.m2/repository/org/apache/logging/log4j/log4j-slf4j-impl/2.17.2/log4j-slf4j-impl-2.17.2.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: Found binding in [jar:file:~/.m2/repository/ch/qos/logback/logback-classic/1.2.10/logback-classic-1.2.10.jar!/org/slf4j/impl/StaticLoggerBinder.class]
SLF4J: See http://www.slf4j.org/codes.html#multiple_bindings for an explanation.
SLF4J: Actual binding is of type [org.apache.logging.slf4j.Log4jLoggerFactory]
```

The above multiple bindings were resolved by excluding log4j-slf4j-impl from org.apache.spark dependency. The dependency was identified by running

> mvn dependency:tree

JIRA: [[ https://yugabyte.atlassian.net/browse/DB-11457 | DB-11457 ]]

Test Plan: Run the UTs and verified that the log messages are present.

Reviewers: mkumar, rbarigidad, nkumar

Reviewed By: rbarigidad

Subscribers: yugaware

Differential Revision: https://phorge.dev.yugabyte.com/D35319
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vpatibandla-yb vpatibandla-yb

Labels
area/ysql Yugabyte SQL (YSQL) jira-originated kind/new-feature This is a request for a completely new feature priority/medium Medium priority issue status/awaiting-triage Issue awaiting triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@yugabyte-ci @vpatibandla-yb