[jboss-dev-forums] [Design of Messaging on JBoss (Messaging/JBoss)] - StorageManager::storeDelete

[timfox]

This method is designed for deleting message ids (see the signature - it says message id).

But it's currently also being used to delete paging related stuff:


  | public void clearLastPageRecord(final LastPageRecord lastRecord) throws Exception
  |    {
  |       trace("Clearing lastRecord information " + lastRecord.getLastId());
  | 
  |       storageManager.storeDelete(lastRecord.getRecordId());
  |    }
  | 

Please don't muddy the separation of concerns here. If you need to delete paging related things please add a different method to do that explicitly.

I notice you did this for the transactional case already, but not the non transactional case


View the original post : http://www.jboss.com/index.html?module=bb&op=viewtopic&p=4195291#4195291

Reply to the post : http://www.jboss.com/index.html?module=bb&op=posting&mode=reply&p=4195291