miguelAngelOjeda / hibernate-generic-dao

Automatically exported from code.google.com/p/hibernate-generic-dao
0 stars 0 forks source link

Add paging to searchUnique() #80

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
See 
http://groups.google.com/group/java-generic-dao/browse_thread/thread/755ae349c5c
46d73?hl=en

Original issue reported on code.google.com by dwolvert on 5 Jan 2011 at 4:11

GoogleCodeExporter commented 9 years ago
See attached patch.

Original comment by terciofi...@gmail.com on 5 Jan 2011 at 4:57

Attachments:

GoogleCodeExporter commented 9 years ago
The patch is actually broken, I'll send another one latter.

Original comment by terciofi...@gmail.com on 5 Jan 2011 at 5:11

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 25 Feb 2011 at 1:55

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 25 Feb 2011 at 2:01

GoogleCodeExporter commented 9 years ago
I just ran into this same issue. Any progress been made?

Original comment by yowza...@gmail.com on 27 Sep 2011 at 7:50

GoogleCodeExporter commented 9 years ago
Hi, for instance you can just ignore setting the paging and get the first 
result.

I could not correct my patch(And now it need some refactoring as the packages 
names has changed). I hope I have time to do some patches to Generic DAO only 
in the next month.

I'm not aware of Wolvert's availability.

Original comment by terciofi...@gmail.com on 27 Sep 2011 at 3:48

GoogleCodeExporter commented 9 years ago
My workaround is to just set max to 1 and do a regular search instead of 
searchUnique. Then I simply grab the first result from the result set.

I just wanted to add my vote to this issue, as it really seems like 
setMaxResults(1) should allow searchUnique to be called without getting a 
NonUniqueException (or whatever it is called).

Original comment by yowza...@gmail.com on 27 Sep 2011 at 6:25

GoogleCodeExporter commented 9 years ago
I'll try to work on this issue...

Original comment by terciofi...@gmail.com on 27 Sep 2011 at 6:35

GoogleCodeExporter commented 9 years ago
Tercio,

You working on this?

I think I'll try to cut a release soon. If you think you'll have this done 
soon, I'll wait for it.

Thanks

Original comment by dwolvert on 2 Oct 2011 at 12:02

GoogleCodeExporter commented 9 years ago
David, Yes, tomorrow I think I finish the test cases.

Original comment by terciofi...@gmail.com on 2 Oct 2011 at 12:13

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
David, see patch attached. Thanks.

Original comment by terciofi...@gmail.com on 2 Oct 2011 at 2:28

Attachments:

GoogleCodeExporter commented 9 years ago

Original comment by dwolvert on 9 Oct 2011 at 5:41