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.

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

[WFLY-18009] Add jars needed for compression of Kafka #16826

Merged
merged 1 commit into from
Nov 2, 2023

Conversation

kabir
Copy link
Contributor

@kabir kabir commented May 10, 2023

@github-actions github-actions bot added the deps-changed Dependencies have been checked, and there are changes highlighted in a comment label May 10, 2023
@github-actions
Copy link

Dependency Tree Analyzer Output:

New Dependencies:

  • com.github.luben:zstd-jni:jar:1.5.2-1:compile
  • org.lz4:lz4-java:jar:1.8.0:compile
  • org.xerial.snappy:snappy-java:jar:1.1.8.4:compile

CC @wildfly/prod

Copy link
Member

@xstefank xstefank left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just a few year fixes and why do you need a logging dependency for the new modules?

@kabir
Copy link
Contributor Author

kabir commented May 11, 2023

@xstefank One of them used logging in the source code, but really it was just an old habit. I guess the log manager stuff we do on boot works anyway, and whatever was using logging was using JUL so perhaps it will work without.

What did you mean by 'year'? :-)

@kabir
Copy link
Contributor Author

kabir commented May 11, 2023

@xstefank it worked locally without the logging dependencies, so I have removed them

@kabir kabir marked this pull request as ready for review July 17, 2023 14:27
@bstansberry bstansberry added 30.x and removed 30.x labels Jul 17, 2023
@bstansberry
Copy link
Contributor

/retest

1 similar comment
@bstansberry
Copy link
Contributor

/retest

@kabir
Copy link
Contributor Author

kabir commented Oct 30, 2023

@bstansberry I fixed the compile error locally after a rebase. Should be fine now

@bstansberry bstansberry merged commit 86b93af into wildfly:main Nov 2, 2023
15 checks passed
@bstansberry
Copy link
Contributor

Thanks @kabir and @xstefank

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-changed Dependencies have been checked, and there are changes highlighted in a comment
Projects
None yet
3 participants