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

RedisSink does not work with Jet 4.1.1 #83

Open
jerrinot opened this issue Jul 1, 2020 · 2 comments
Open

RedisSink does not work with Jet 4.1.1 #83

jerrinot opened this issue Jul 1, 2020 · 2 comments
Assignees
@gurbuzali
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@jerrinot
Copy link
Contributor

jerrinot commented Jul 1, 2020 

SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details.
Exception in thread "main" java.lang.NoSuchMethodError: 'void com.hazelcast.jet.impl.util.Util.checkSerializable(java.lang.Object, java.lang.String)'
	at com.hazelcast.jet.contrib.redis.RedisSinks.hash(RedisSinks.java:141)
	at com.hazelcast.jet.examples.callhomes.CountDistinctVersionsHomes.main(CountDistinctVersionsHomes.java:51)
@jerrinot jerrinot added bug Something isn't working good first issue Good for newcomers labels Jul 1, 2020
@jerrinot
Copy link
Contributor Author

jerrinot commented Jul 1, 2020

It was caused by this change: hazelcast/hazelcast-jet@cc3a51a#diff-0309b636066cb39dd2e80cf494db06e3R171 which is not binary compatible (a different return type)

It's hard to blame this commit though as it's an internal class.

@viliam-durina
Copy link
Contributor

Contributed sinks should not use private API...

@gurbuzali gurbuzali self-assigned this Jul 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gurbuzali gurbuzali

Labels
bug Something isn't working good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@jerrinot @gurbuzali @viliam-durina