History | Log In     View a printable version of the current page. Get help!  
Issue Details [XML]

Key: SES-543
Type: Improvement Improvement
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: James Leigh
Reporter: Arjohn Kampman
Votes: 0
Watchers: 0

If you were logged in you would be able to see more operations.

Move SailConnectionListener functionality to an optional interface

Created: 16/Apr/08 04:21 PM   Updated: 26/Sep/08 11:33 AM
Component/s: Sail API
Affects Version/s: None
Fix Version/s: 2.2

The SailConnectionListener functionality that is offered in SailConnection poses severe problems for some Sail implementations and should therefore be moved to a separate interface.

Also, the per-statement update notification scheme doesn't scale very well. For example, clearing all data will trigger a huge amount of events that can probably be handled more efficiently. Alternative (trigger-based?) options should be considered.

 All   Comments   Change History      Sort Order:
Change by Arjohn Kampman [24/Apr/08 04:00 PM]
Field Original Value New Value
Fix Version/s 2.2 [ 10330 ]
Fix Version/s 2.1 [ 10240 ]

Change by James Leigh [07/May/08 04:30 PM]
Field Original Value New Value
Assignee Arjohn Kampman [ arjohn ] James Leigh [ james ]

Comment by James Leigh [07/May/08 08:47 PM]
Created NotifyingSail and NotifyingsailConnection
Changed Sail*Base and Sail*Wrapper to implement the corresponding interfaces
revision 6914

Change by James Leigh [07/May/08 08:48 PM]
Field Original Value New Value
Status Open [ 1 ] Resolved [ 5 ]
Fix Version/s 2.1.1 [ 10331 ]
Fix Version/s 2.2 [ 10330 ]
Resolution Fixed [ 1 ]

Change by Arjohn Kampman [08/May/08 12:48 PM]
Field Original Value New Value
Fix Version/s 2.2 [ 10330 ]
Fix Version/s 2.1.1 [ 10331 ]

Comment by Arjohn Kampman [22/May/08 04:16 PM]
Moved notification functionality from default wrappers to new notifying wrapper subclasses, svn rev 7191.

Change by Arjohn Kampman [26/Sep/08 11:33 AM]
Field Original Value New Value
Status Resolved [ 5 ] Closed [ 6 ]