My favorites | Sign in
Project Home Downloads Wiki Issues Source
New issue   Search
for
  Advanced search   Search tips   Subscriptions
Issue 251: enable/disable timeout does the opposite of what it says
2 people starred this issue and may be notified of changes. Back to list
Status:  Fixed
Owner:  google-a...@codesmell.de
Closed:  Mar 2011


Sign in to add a comment
 
Project Member Reported by google-a...@codesmell.de, Mar 21, 2011
probably, it should be called enableNoTimeout or the logic should be reversed.
Mar 21, 2011
Project Member #1 scotthernandez
We might want to call it a cursor timeout since that is more accurate. There may be other timeouts in the future.
Mar 21, 2011
Project Member #2 google-a...@codesmell.de
it would be a breaking API change then?

point is, it was broken. i added enableCursorTimeout and deprecated the old ones (while flipping their behaviour). please feel free to remove the deprecated ones, if you think it is appropriate.
Status: Fixed
Mar 21, 2011
Project Member #3 google-a...@codesmell.de
oh. too late, you already did.
May 29, 2011
#4 joeym...@gmail.com
In which version of Morphia might we expect to see this fix?
May 29, 2011
Project Member #5 scotthernandez
(No comment was entered for this change.)
Labels: Milestone-Release1.0
Nov 12, 2011
Project Member #6 scotthernandez
(No comment was entered for this change.)
Labels: Milestone-Release0.99.1
Sign in to add a comment

Powered by Google Project Hosting