wood burning stoves 2.0*
The moose likes Developer Certification (SCJD/OCMJD) and the fly likes URLyBird: Checking for a deleted record in a Lock Manager Big Moose Saloon
  Search | Java FAQ | Recent Topics | Flagged Topics | Hot Topics | Zero Replies
Register / Login


Win a copy of Murach's Java Servlets and JSP this week in the Servlets forum!
JavaRanch » Java Forums » Certification » Developer Certification (SCJD/OCMJD)
Bookmark "URLyBird: Checking for a deleted record in a Lock Manager " Watch "URLyBird: Checking for a deleted record in a Lock Manager " New topic
Author

URLyBird: Checking for a deleted record in a Lock Manager

Daniel Bryant
Ranch Hand

Joined: Jul 15, 2006
Posts: 54
Hi everyone,

I'm currently implementing my Data class for URLyBird 1.2.1. There will be a single instance of the Data class that all clients will connect to via RMI. To promote extensibility and make it easier for other (junior?) programmers to maintain my code I have implemented the Data class using the Facade design pattern, with each method that must be implemented delegating appropriately to either a DBFileAccess class or a LockManager class (i.e. create(), update() etc to DBFileAccess and lock(), unlock() to LockManager.

The DBFileAccess class completely encapsulates the DB file access and has no connection to the logical record locking - exactly what we are after. However, the problem I have is that the lock() method of my LockManager class must check (within a synchronized block) if a record has been deleted while the current thread has been waiting for the lock.



Currently I have decided to pass the instance of DBFileAccess into the LockManager class so that I can call isDeletedRec() within the synchronized lock() method, but this means my LockManager knows about the DatabaseFileAccess class, which kind of defeats the objective of having two seperate classes with only one responsibility each.

Can anyone offer any suggestions of how I could overcome this problem (and I can't move the delete check to the calling class as I would exit the synchronized block), OR do you think that the LockManager lock() method making one call to the DB class is acceptable (and it's not a big deal that the LockManager is reliant on a method within the DBFileAccess class)?

Your thoughts would be greatly appreciated.

Daniel


SCJP 1.4,<br />SCJD (URLyBird 1.2.1, JDK 5.0)
Oricio Ocle
Ranch Hand

Joined: Nov 30, 2004
Posts: 284

Hello Daniel,
your design seems good.
Only one suggestion: Isolate completely lockmanager implementation from data.

In your case my position would be:
DBFileAccess: CRUD operations over both, deleted and not-deleted records.
LockManager: Locking methods in order to ensure logical consistency of abstract indexed entities (records in your implementation)
Data: Makes use of these, and implements the specified interface. Here is were I would verify record existence:

Data:lockRecord


Regards


SCJP, OCMJD, OCMJEA
Daniel Bryant
Ranch Hand

Joined: Jul 15, 2006
Posts: 54
Hi Oricio,

Many thanks for your feedback and excellent suggestion. I guess I missed the fact that I could do the test for a deleted record in my facade Data class. And as you said, it definitely makes sense to cleanly seperate the DB access and locking responsibility - when I was calling the DB in my LockManager alarm bells started ringing!!

Once again, many thanks.

Daniel
Harish Yerneni
Ranch Hand

Joined: Sep 15, 2004
Posts: 94
Hello Ranchers:

I have the same problem as Daniel (but in a B&S assignment).

I am not sure if I understood orcio's answer correctly. I didnt understand why she unlock in a lockrecord call. After reading the post, this is what I did in my lock method in Data class (facade). Is that what he/she meant or is it appropriate?



Thanks,

Harish


SCJP 1.4 | SCWCD 1.4 | SCJD (WIP)
Herman Schelti
Ranch Hand

Joined: Jul 17, 2006
Posts: 387
hi all,

does your spec really require that the lock method checks if a record is deleted on not?

And if the answer is "yes": if you check the existence in the database file first, and after that lock the record, there still is a chance that the record got deleted in between...

Herman
Cristian Aionesa
Greenhorn

Joined: Jan 05, 2007
Posts: 20
Hi to all,

I have a problem regarding throwing RecordNotFoundException. My Data class is a facade, just like yours, and I use two other classes DataAccessManager and LockingManager for the actual work. I'm not sure if RecordNotFoundException should be thrown by the DataAccessManager object and allowed by Data to propagate on the stack, or if Data should do something like this:



I like this approach more, but I'm not sure if this design doesn't contradict the idea behind facade pattern.
Any suggestions are welcomed
Roel De Nijs
Bartender

Joined: Jul 19, 2004
Posts: 5126
    
  12

Hi Christian,

Don't wake up zombie threads! It has been more than 2 years since this thread was active. The last poster in this thread posted his last message more than 1 year ago, the original poster posted his last contribution more than 2 years ago.

Just start a new thread with your problem/question and other ranchers (like me) will be glad to help you.

Kind regards,
Roel


SCJA, SCJP (1.4 | 5.0 | 6.0), SCJD
http://www.javaroe.be/
Cristian Aionesa
Greenhorn

Joined: Jan 05, 2007
Posts: 20
Ok, sure. I didn't know about zombie threads, my bad

Thanks!
 
It is sorta covered in the JavaRanch Style Guide.
 
subject: URLyBird: Checking for a deleted record in a Lock Manager
 
Similar Threads
URLybird
pls validate my locking strategy - all inputs are g8ly appreciated. (URLyBird)
Locking strategy URLyBird 1.1.1 with ReentrantLock
URLyBird Locking
[URLyBird]Is my lock appraoch right?