• Post Reply Bookmark Topic Watch Topic
  • New Topic
programming forums Java Mobile Certification Databases Caching Books Engineering Micro Controllers OS Languages Paradigms IDEs Build Tools Frameworks Application Servers Open Source This Site Careers Other Pie Elite all forums
this forum made possible by our volunteer staff, including ...
Marshals:
  • Campbell Ritchie
  • Jeanne Boyarsky
  • Ron McLeod
  • Paul Clapham
  • Liutauras Vilda
Sheriffs:
  • paul wheaton
  • Rob Spoor
  • Devaka Cooray
Saloon Keepers:
  • Stephan van Hulst
  • Tim Holloway
  • Carey Brown
  • Frits Walraven
  • Tim Moores
Bartenders:
  • Mikalai Zaikin

Appealing grading results?

 
Greenhorn
Posts: 26
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator
Hi,
Is it possible to appeal the grading result?

:-) yes, I failed. Zero points in locking. But, I don't think the
tester is right. The explanation - "Your locking method does not block when trying to lock a locked record." - is not true.
Pretty arrogant from me, hmm? :-)
I believe what I see in the code and it is a classic wait-pattern.

What should I do :-)
Greetings
Jaro
 
Ranch Hand
Posts: 288
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator

Originally posted by Jaro Prospal:
Hi,
Is it possible to appeal the grading result?

:-) yes, I failed. Zero points in locking. But, I don't think the
tester is right. The explanation - "Your locking method does not block when trying to lock a locked record." - is not true.
Pretty arrogant from me, hmm? :-)
I believe what I see in the code and it is a classic wait-pattern.

What should I do :-)
Greetings
Jaro




Can you put up your locking methods?
 
Ranch Hand
Posts: 65
  • Mark post as helpful
  • send pies
    Number of slices to send:
    Optional 'thank-you' note:
  • Quote
  • Report post to moderator

The explanation - "Your locking method does not block when trying to lock a locked record." - is not true.
Pretty arrogant from me, hmm? :-)
I believe what I see in the code and it is a classic wait-pattern.



Did you test your locking?
You can write a test code in which:
  • one thread calls the lock a record.
  • then, another thread tries lock the same record (when the first thread hasn't called unlock). And you must see the status of this thread (using getState() method) to be WAITING.

  • If it still works fine, then, you can contact Sun.

    Good luck.
    Chulwoo
     
    Jaro Simak
    Greenhorn
    Posts: 26
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Thanks for your interest.
    Well, maybe I am going to blame myself, but after all I did it already :-)


    Here is the piece from Data class.
    The synchronizes on the member lockedRecords. If it gets the monitor, it checks if the record
    is locked. The method waitUntilRecordIsNotLocked returns true if the record is unlocked and
    false is already locked by the same thread (for instance by calling the lock method twice).
    I have to say, the implementation of the sequence lock, update, unlock reside on the server
    ("thin" client), so I identify the clients by the thread reference.
    Data class is singleton.



    I really don't know what is there wrong.

    [Andrew: put code between [code] and [/code UBB tags]
    [ November 09, 2006: Message edited by: Andrew Monkhouse ]
     
    Mark Smyth
    Ranch Hand
    Posts: 288
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator

    Originally posted by Jaro Prospal:
    Thanks for your interest.
    Well, maybe I am going to blame myself, but after all I did it already :-)


    Here is the piece from Data class.
    The synchronizes on the member lockedRecords. If it gets the monitor, it checks if the record
    is locked. The method waitUntilRecordIsNotLocked returns true if the record is unlocked and
    false is already locked by the same thread (for instance by calling the lock method twice).
    I have to say, the implementation of the sequence lock, update, unlock reside on the server
    ("thin" client), so I identify the clients by the thread reference.
    Data class is singleton.



    I really don't know what is there wrong.

    [Andrew: put code between [code] and [/code UBB tags]

    [ November 09, 2006: Message edited by: Andrew Monkhouse ]



    Must say if found your code a slightly more complicated than I think it needed to be, and you should probably also have a notifyAll() at the end of the lock method to inform other threads that you are finished with the monitor.

    Having said all that I tried your code (with a few tweaks to get it running cos I dont have all your code ie the unlock methods). I ran 6 threads calling lock(0) without unlocking and got the following output. As you can see there are five threads waiting patiently forever to get a lock that will never be released.



    I ran this code and added println statements to your methods;


    Without having seen all the implementation I think that you definately have grounds for an appeal as it definately blocks other threads while the record is locked. 0 / 40 is an unfair score and the reason given for it is false, so go for it.

    Best of luck with it,

    Mark

    [ November 10, 2006: Message edited by: Mark Smyth ]
    [ November 10, 2006: Message edited by: Mark Smyth ]
     
    Jaro Simak
    Greenhorn
    Posts: 26
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Thank you very much for spending so much time with the problem. Thanks for the putting the UBB Codes inside the posts as well. Next time I will take care of it.

    It is good to hear that I am right.
    It took 10 week to grade the assignment, anyway. And for this result
     
    author and jackaroo
    Posts: 12200
    280
    Mac IntelliJ IDE Firefox Browser Oracle C++ Java
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Hi Jaro,

    Sorry - ran out of time yesterday, so didn't respond (and now I am running late for work, so this will have to be short as well).

    I think you do have reason to appeal the decision, as you are blocking where you need to.

    Technically the assignment does not suggest that you should allow the same record to be locked twice by the same client. So I guess if someone wanted to be pedantic about it, they could claim that you are not blocking in that case. But if you did, you would create a scenario where a lock could never be released. So I think your way of handling this is a good design decision.

    It looks like your thread will get the mutex on the LockMap object twice while performing a lock operation - while this wont cause a problem, it is generally not desirable.

    But none of those should cause a total loss of marks.

    If you do decide to appeal, be aware that in the past we have seen that it takes quite a while to get the appeal processed, and you may have to send multiple emails to ensure that it continues to be worked on. Good luck with it though.

    Originally posted by Mark Smyth:
    [...] and you should probably also have a notifyAll() at the end of the lock method to inform other threads that you are finished with the monitor. [...]

    There's no need for that - you only need to notify other threads when there is a reason for them to re-check whether their lock is now available - in other words, at the end of your unlock() method.

    Regards, Andrew
     
    Ranch Hand
    Posts: 1847
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Allowing a thread to lock its own locked records again might indeed be the cause of the failure you got.
    Technically the assessor is of course correct there, if he wants to be extremely strict.
    I think throwing an exception ("attempting to lock a record already locked by this thread" or some such) would have been better.

    Personally I'd not have rejected a solution based on that alone, but I don't have the guidelines Sun's assessors have to work to at my disposal (of course).
     
    Ranch Hand
    Posts: 226
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    I'm not yet working on my SCJD but I did also get your code working after filling in the gaps (of course I didn't have your other classes). So baring in major differences there, It also worked for me.

    As to the thread relocking an already locked record. Unless its in the spec about what to do with that, I like your solution. Whats the harm in trying to relock something already locked? Why bother with an exception?

    But from the explanation - "Your locking method does not block when try to lock a locked record" - that is indeed true if its the same thread - but that seems undesirable to me - as how would it ever get unlocked?

    I agree with no need for notifyAll at the end of lock. There isn't a reason for the other threads to check locks at this point - nothing is newly unlocked.

    Good luck!
     
    Jaro Simak
    Greenhorn
    Posts: 26
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Hi,
    thanks a lot for your interest.

    My assignment (B&S 2.2.3) defines the lock method as follows:



    The reason that I prohibited to lock a record that is already locked by the same thread was the formulation "different thread". I guess the implementation is correct.


    It looks like your thread will get the mutex on the LockMap object twice while performing a lock operation - while this wont cause a problem, it is generally not desirable.


    Yes, the synchronized modifier in the method waitUntilRecordIsNotLocked
    is an overhead, it is not necessary.

    Greetings
    Jaro
    [ November 11, 2006: Message edited by: Jaro Prospal ]
     
    Ranch Hand
    Posts: 918
    IntelliJ IDE Java Linux
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Hi, People

    I just read the post and I hope I not to late with my reply.
    I think that the main problem is the "unique client id",
    In the code snippet you can see that the Lock Manager uses the pair [cookie, Thread]. This logic may work if every client has only one thread but if we deal with RMI we lose the control over the threads, there is no guarante
    e that every client will use a separate thread(this theme was enough discussed on the ranch).
    This only if the RMI is used.

    So lets say that the RMI daemon uses only one thread(only hypothetic) to solve its remote requests. Will be something like :


    Three clients (with three different threads) send request to the RMI . The RMI uses a single thread to solve all this requests.
    So if client 1 locks record1, the Lock manager stores [record 1, RMI Thread]
    if client 2 locks the record 1, the lock manager think that the same thread (the RMI Thread) and it returns false and this is wrong because the Record 1 is already locked (from Client 1).
    If you do a simple multithread test (like Mark suggest) where every client will have the same thread, the test will be successful.

    Regards M.
    [ November 15, 2006: Message edited by: Mihai Radulescu ]
     
    Jaro Simak
    Greenhorn
    Posts: 26
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Hi Radu,
    yes, you right and I was thinking about this case.

    But,in my case, all the lock, update, unlock logic resides on the server. Some folks here call it "thin" client.
    Thus, a thread processes the three calls for one request and then can go to process other ones.

    On the other side, the testers can test the code the way you described - remote calls of lock and unlock from
    clients. In that case, it may happen that the code does not work, as you described. But, these tests seems to
    me against the claim in the assignment:


    You may assume that at any moment, at most one program is accessing the database file;
    therefore your locking system only needs to be concerned with multiple concurrent clients of your server.


    The tests can be considered as another server and for that server the locking mechanism is not required to work.

    Another point is, I have no idea how to transport some client IDs through the API.

    Greeting
    Jaro
     
    Ranch Hand
    Posts: 89
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator

    remote calls of lock and unlock from
    clients.



    I hope they don't do this. In real application, this is a disaster. I am going to discuss this in choices.txt to make sure they understand that.
     
    Mark Smyth
    Ranch Hand
    Posts: 288
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator

    Originally posted by Jaro Prospal:
    Hi Radu,
    yes, you right and I was thinking about this case.

    But,in my case, all the lock, update, unlock logic resides on the server. Some folks here call it "thin" client.
    Thus, a thread processes the three calls for one request and then can go to process other ones.

    On the other side, the testers can test the code the way you described - remote calls of lock and unlock from
    clients. In that case, it may happen that the code does not work, as you described. But, these tests seems to
    me against the claim in the assignment:

    The tests can be considered as another server and for that server the locking mechanism is not required to work.

    Another point is, I have no idea how to transport some client IDs through the API.

    Greeting
    Jaro



    I think that your thin client approach is a valid one (which I am also using) and has been used in the past by others succesfully to pass the SJCD. To be honest I think the reason you failed is because the examiner did not understand (or look at) your wait method in the code or else did not investigate it in enough detail. His comments when failing you only mentioned that the lock method did not block which it clearly does and he made no mention of any other threading issue. I would go ahead with the appeal and I am certain that you will be passed on reconsideration. Good luck and don't take no for an answer.

    Mark
     
    Mihai Radulescu
    Ranch Hand
    Posts: 918
    IntelliJ IDE Java Linux
    • Mark post as helpful
    • send pies
      Number of slices to send:
      Optional 'thank-you' note:
    • Quote
    • Report post to moderator
    Jaro,

    You don't need to transport the client unique id.
    You can use a remote factory - this theme was to much discussed here, just do a search using remote factory or peek in the Andrew book - he uses the same "trick".

    Mark,

    You do some serious assumption here,


    has been used in the past by others succesfully to pass the SJCD


    I don't think that "thread like id" has ever a chance to pass the SJCD.
    You can try and risk.

    Regards M.
     
    Not so fast naughty spawn! I want you to know about
    a bit of art, as a gift, that will fit in a stocking
    https://gardener-gift.com
    reply
      Bookmark Topic Watch Topic
    • New Topic