-
Notifications
You must be signed in to change notification settings - Fork 539
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
Secor v0.28 is released #1398
Comments
@HenryCaiHaiying Thanks for releasing v0.28 , just now i tried to build jars using the codebase of v0.28 tag release but getting this
|
i am on macOS, with the following versions, if that helps
|
I am not sure if I am missing something (it could very likely be, as i am new to java and trying to build the jars). Alternately if you have 0.28, jars that you can release or a Dockerfile that i can use. Any support would be greatly appreciated |
Are you just doing 'mvn clean package"? I have an Linux env, I didn't see problems. |
I synced the code to head (not on v0.28 tag) |
@ksingh7 looks like you problem might be related to your java version (14), looks like this issue with MockClassLoader starts with java 9: https://stackoverflow.com/questions/50456726/mockclassloader-cannot-access-jdk-internal-reflect-superclass-jdk-internal-refle |
@HenryCaiHaiying So i tried this
but this time as well it failed with exact same error
What is your java/mvn version on your Linux workstation, may be i can give it a try on a Linux box. Any other suggestion you may have for me ? |
@HenryCaiHaiying and as per this comment can you please guide me on what test base files i need to add the mentioned annotation ? |
So there are a lot of test files, where i need to add
Is there a base class, where i can change it once and inherited on all test files ? |
java8 definitely works, I am also trying the suggestions from that stack overflow with a java11 build. |
After trying this comment
|
@HenryCaiHaiying i have tried a lot with no success, thank a TON for your help so far. I will go and take some rest, will try again tomorrow. If you have any tips for me let me know |
@HenryCaiHaiying confirming that build works on java8 (Linux) , flawlessly |
Yes, java8 works. I am still working on java11, fixed a few unit tests but still have a few more to go. |
This PR should be able to resolve unit test failures in JDK 11: #1400 |
@HenryCaiHaiying you are awesome, i will give it a try. |
It's been a year since v0.27 is released. In v0.28, we upgrade the default build profile to kafka-2.0 and include various bug fixes.
Please take a look at https://github.com/pinterest/secor/blob/master/RELEASE.md to see the list of bug fixes.
The text was updated successfully, but these errors were encountered: