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

jsonDBTemplate.findAll(Instance.class, comparator) #46

Open
EclipseKnight opened this issue Jan 21, 2021 · 4 comments
Open

jsonDBTemplate.findAll(Instance.class, comparator) #46

EclipseKnight opened this issue Jan 21, 2021 · 4 comments

Comments

@EclipseKnight
Copy link

This method is documented on the site but is not usable.
image of IDE
Am I missing something?

@BlueDrink9
Copy link

This is because you are using the released version, which doesn't have the features documented on the project homepage. @FarooqKhan any chances of getting a new tag/release so that tools like maven can access the new jsondb features?

@FarooqKhan
Copy link
Collaborator

This method has always been part of the released jar. I downloaded the jar from mvnrepository and decompiled and checked. It shows up in the jar. I have had this coded for over 5 years now. There has to be some other problem.

@BlueDrink9
Copy link

This functionality didn't work for me either, until I switched from the tagged release to the latest git commit.

I feel wrong telling you that you are incorrect about your own code, but if you look at the class as of the release commit you'll see that actually, those method overloads don't exist in the class as documented. Assuming the release jar comes from that commit (and it would be bafflingly poor practise if it did not), then the release jar does not have this feature.

@BlueDrink9
Copy link

There have been 100+ commits since the last git tag was released, and that seems to be what gets picked up by Maven

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants