added @Overrides and import fixes #212

Closed
wants to merge 1 commit into
from

3 participants

@raszi

There are missing @Override annotations in Jedis code, I fixed them in this patch.

@xetorthio
Owner

I remember removing them because of compatibility issues between java 6 and java 5.
But I just checked javadocs and it seems like it should be compatible. So I can't really remember why it was complaining.
Anyway, if it works when compiled with java 5, we should merge this.

@raszi

Java 6 needs even more @Override annotations. These are only for Java 5.

@HeartSaVioR
Collaborator

I agree that we should add @Override annotations to Jedis, but many things were changed so current PR seems to be invalid.
So I'm closing this issue and open new issue addressing on this. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment