不要根据MemcacheClient的delete结果来决定是否删除数据库记录
2012-09-27 22:41
274 查看
今天在给同事做CodeReview时,发现了一个问题。代码大致如下:
1.Memcache客户端采用的gwhalin / Memcached-Java-Client v2.6.3:https://github.com/gwhalin/Memcached-Java-Client
2.问题分析:
若对应的key在缓存服务器中已经不存在,或者memCachedClient.get(key)返回值为null,那么delete()方法的返回值为false。试想,如果缓存过期、或被算法逐出、或因故障缓存数据丢失等等情况,这些情况发生时memCachedClient.get(key)都会返回null,memCachedClient.delete(key)都会返回false,所以以上代码块中的delete stuffs from DB分支都无法执行。
所以根据delete的返回值来决定是否进行DB中数据的删除,那么可能会导致DB中的数据在一段时间内甚至永远得到删除,影响数据一致性。
3.改进:
delete之前先get一下,若为null,则认为删除成功。
还未来得及看gwhalin / Memcached-Java-Client v2.6.3的源码,如果get为null,delete返回值到底如何处理的,后续再更新
to be continued....
>>>>>>>>>>更新<<<<<<<<<<<
查看MemcachedClient源码public boolean delete( String key, Integer hashCode, Date expiry )部分,当server端返回“NOT_FOUND”时,未做任何处理,最后返回false。
else if ( NOTFOUND.equals( line ) ) {
if ( log.isInfoEnabled() )
log.info( "++++ deletion of key: " + key + " from cache failed as the key was not found" );
}
public boolean delete(String key){ boolean isCacheDeleted = memcachedClient.delete(key); if(isCacheDeleted){ //delete stuffs from DB } //other stuffs }
1.Memcache客户端采用的gwhalin / Memcached-Java-Client v2.6.3:https://github.com/gwhalin/Memcached-Java-Client
2.问题分析:
若对应的key在缓存服务器中已经不存在,或者memCachedClient.get(key)返回值为null,那么delete()方法的返回值为false。试想,如果缓存过期、或被算法逐出、或因故障缓存数据丢失等等情况,这些情况发生时memCachedClient.get(key)都会返回null,memCachedClient.delete(key)都会返回false,所以以上代码块中的delete stuffs from DB分支都无法执行。
所以根据delete的返回值来决定是否进行DB中数据的删除,那么可能会导致DB中的数据在一段时间内甚至永远得到删除,影响数据一致性。
3.改进:
delete之前先get一下,若为null,则认为删除成功。
public boolean delete(String key){ boolean isCacheDeleted = false; Object obj = memcachedClient.delete(key); if(obj == null){ isCacheDeleted = true; }else{ isCacheDeleted = memcachedClient.delete(key); } if(isCacheDeleted){ //delete stuffs from DB } //other stuffs.... }4.其他
还未来得及看gwhalin / Memcached-Java-Client v2.6.3的源码,如果get为null,delete返回值到底如何处理的,后续再更新
to be continued....
>>>>>>>>>>更新<<<<<<<<<<<
查看MemcachedClient源码public boolean delete( String key, Integer hashCode, Date expiry )部分,当server端返回“NOT_FOUND”时,未做任何处理,最后返回false。
else if ( NOTFOUND.equals( line ) ) {
if ( log.isInfoEnabled() )
log.info( "++++ deletion of key: " + key + " from cache failed as the key was not found" );
}
相关文章推荐
- 用SqlDataAdapter.DeleteCommand根据DataSet删除数据库记录的失败教训
- mysql之delete删除记录后数据库大小不变
- mysql之delete删除记录后数据库大小不变
- 如果把记录从数据库里面删除,判断是否重复?
- mysql delete删除记录数据库空间不减少问题解决方法
- IOS开发中如何判断程序第一次启动(根据判断结果决定是否显示新手操作引导)
- 数据库记录的删除,delete好还是update好?
- 数据库记录的删除,delete好还是update好?
- 把数据写入到数据库的时,常常会碰到先要检测要插入的记录是否存在,然后决定是否要写入(转)
- mysql delete删除记录数据库空间不减少问题解决方法
- mysql之delete删除记录后数据库大小不变
- ElasticSearch中根据查询结果删除数据(delete by query)
- mysql之delete删除记录后数据库大小不变
- SQL SERVER快速删除数据库表里面的记录,truncate与delete的对比 【转】
- mysql之delete删除记录后数据库大小不变
- SQL Server 2005数据库定期备份和删除历史备份记录
- 千万级数据库(MSSQL)删除重复记录
- 是否有重复记录并删除
- mysql 删除一个不确定是否存在的数据库
- mysql支持跨表delete删除多表记录