这令我感到困惑。我搜索了很多论坛的线索。在postFlush方法中,我使用一个独特的会话工厂打开一个新的hibernate(v.3.3 - 也尝试过v.3.5)会话。我使用c3p0 v.9.0进行连接池。我启动一个新事务,保存auditLog对象,并提交事务。这对我的所有实体都很好用。在删除ChemoRegimen实体后尝试提交auditLog时,应用程序挂起(创建和更新也会发生这种情况)。没有抛出异常,但在挂起线程后,我发现以下堆栈跟踪(这是一个Swing应用程序):
Thread [AWT-EventQueue-0] (Suspended)
SocketInputStream.socketRead0(FileDescriptor, byte[], int, int, int) line: not available [native method]
SocketInputStream.read(byte[], int, int) line: 129
VisibleBufferedInputStream.readMore(int) line: 145
VisibleBufferedInputStream.ensureBytes(int) line: 114
VisibleBufferedInputStream.read() line: 73
PGStream.ReceiveChar() line: 274
QueryExecutorImpl.processResults(ResultHandler, int) line: 1660
QueryExecutorImpl.execute(Query[], ParameterList[], ResultHandler, int, int, int) line: 407
Jdbc4PreparedStatement(AbstractJdbc2Statement).executeBatch() line: 2737
NewProxyPreparedStatement.executeBatch() line: 1723
BatchingBatcher.doExecuteBatch(PreparedStatement) line: 70
BatchingBatcher(AbstractBatcher).executeBatch() line: 268
ActionQueue.executeActions(List) line: 266
ActionQueue.executeActions() line: 167
DefaultFlushEventListener(AbstractFlushingEventListener).performExecutions(EventSource) line: 321
DefaultFlushEventListener.onFlush(FlushEvent) line: 50
SessionImpl.flush() line: 1027
SessionImpl.managedFlush() line: 365
JDBCTransaction.commit() line: 137 **[This is where I commit the auditLog]**
MomsInterceptor.postFlush(Iterator) line: 254
DefaultFlushEventListener(AbstractFlushingEventListener).postFlush(SessionImplementor) line: 375
DefaultFlushEventListener.onFlush(FlushEvent) line: 51
SessionImpl.flush() line: 1027
SessionImpl.managedFlush() line: 365
JDBCTransaction.commit() line: 137
HibernateDAO.makeTransient(Entity) line: 119
ChemoServices.deleteChemoRegimen(ChemoRegimen, String, Session) line: 290
我使用PostgreSQL 8.4作为9.0 jdbc4驱动程序的后端。 postgresql.log显示[带括号中的注释]:
[首先,删除chemo_regimen]
2011-05-11 12:19:06 CDT moms postgres LOG: 00000: execute <unnamed>: delete from moms_chemo_regimen where crxreg_id=$1<BR>
....
[大量级联删除]
...
[然后,我的交易从拦截器会话开始]
2011-05-11 12:19:06 CDT moms postgres LOG: 00000: execute S_1: BEGIN
2011-05-11 12:19:06 CDT moms postgres LOG: 00000: execute <unnamed>: select nextval ('moms_patient_change_log_seq')
[插入审核日志记录]
2011-05-11 12:19:06 CDT moms postgres LOG: 00000: execute <unnamed>: insert into moms_patient_change_log (patclog_pat_id, patclog_action, patclog_reason, patclog_date, patclog_user_name, patclog_guid, patclog_id) values ($1, $2, $3, $4, $5, $6, $7)
2011-05-11 12:19:06 CDT moms postgres DETAIL: parameters: $1 = '17108', $2 = 'Deleted ChemoRegimen ABVD', $3 = NULL, $4 = '2011-05-11 12:19:06.813', $5 = 'daver', $6 = 'BFAA9D91-7A4E-835E-7A57-B72B2A79A4F1', $7 = '520'
就是这样。审计日志插入事务永远不会完成。删除化疗方案的交易也从未实施过。我可以在不审核时对ChemoRegimen进行CRUD罚款。 ChemoRegimen实体的片段如下:
public class ChemoRegimen extends MOMSEntity implements Auditable
{
public static final String UNSCHEDULED = "UNSCHEDULED";
private Date date = new Date();
private Patient patient;
private WorkingProtocol protocol;
private Physician approvingPhysician;
private boolean canChangeCycles;
private List<ChemoEncounter> chemoEncounters = new ArrayList<ChemoEncounter>();
private boolean complete;
private Icdm icdm;<BR>
...<BR>
}
这是我的拦截器:
public class MomsInterceptor extends EmptyInterceptor
{
private static Logger logger = Logger.getLogger( MomsInterceptor.class.getName() );
private static Configuration configuration;
private static SessionFactory sessionFactory;
//Create the initial SessionFactory from the default configuration files
static
{
initSessionFactory();
}
public static void initSessionFactory()
{
try
{
configuration = new Configuration().configure();
sessionFactory = configuration.buildSessionFactory();
}
catch ( Throwable ex )
{
// We have to catch Throwable, otherwise we will miss
// NoClassDefFoundError and other subclasses of Error
logger.severe( "Building SessionFactory failed - " + ex.getMessage() );
System.err.println( "Building SessionFactory failed - " + ex.getMessage() );
throw new ExceptionInInitializerError( ex.getMessage() );
}
}
private Set<Auditable> inserts = new HashSet<Auditable>();
private Set<UpdatedEntity> updates = new HashSet<UpdatedEntity>();
private Set<Auditable> deletes = new HashSet<Auditable>();
private boolean audit;
public MomsInterceptor(boolean audit)
{
super();
this.audit = audit;
}
private class UpdatedEntity
{
private Auditable auditable;
private String[] propertyNames;
private Object[] currentState;
private Object[] previousState;
private Type[] types;
public UpdatedEntity( Auditable auditable, String[] propertyNames, Type[] types, Object[] currentState, Object[] previousState )
{
super();
this.auditable = auditable;
this.propertyNames = propertyNames;
this.currentState = currentState;
this.previousState = previousState;
this.types = types;
}
public Auditable getAuditable()
{
return auditable;
}
public String[] getPropertyNames()
{
return propertyNames;
}
public Object[] getCurrentState()
{
return currentState;
}
public Object[] getPreviousState()
{
return previousState;
}
public Type[] getTypes()
{
return types;
}
/**
* Return the previous value of the property name prop or null if the property name is not found.
* @param prop
* @return
*/
public Object getPrevious( String prop )
{
int i = 0;
for ( String name : propertyNames )
{
if ( prop.equals( name ) )
return previousState[i];
i++;
}
return null;
}
}
public boolean onSave( Object entity, Serializable id, Object[] state, String[] propertyNames, Type[] types ) throws CallbackException
{
boolean modified = false;
if ( entity instanceof MutableEntity ) // Update create info.
{
MutableEntity me = (MutableEntity)entity;
int i = findPropertyNameIndex( "createUser", propertyNames );
if ( i >= 0 )
state[i] = SessionController.userName;
i = findPropertyNameIndex( "modifyUser", propertyNames );
if ( i >= 0 )
state[i] = SessionController.userName;
modified = true;
if ( audit && entity instanceof Auditable )
inserts.add( (Auditable)entity );
}
return modified;
}
private int findPropertyNameIndex( String name, String[] propertyNames )
{
int i = -1;
if ( propertyNames.length == 0 )
return i;
for ( String p : propertyNames )
{
i++;
if ( p.equals( name ) )
return i;
}
return -1;
}
public boolean onFlushDirty( Object entity, Serializable id, Object[] currentState, Object[] previousState, String[] propertyNames, Type[] types )
throws CallbackException
{
boolean modified = false;
if ( entity instanceof MutableEntity ) // Update modify info.
{
MutableEntity me = (MutableEntity)entity;
int i = findPropertyNameIndex( "modifyUser", propertyNames );
if ( i >= 0 )
currentState[i] = SessionController.userName;
i = findPropertyNameIndex( "modifyDate", propertyNames );
if ( i >= 0 )
currentState[i] = new Date();
modified = true;
if ( audit && entity instanceof Auditable )
updates.add( new UpdatedEntity( (Auditable)entity, propertyNames, types, currentState, previousState ) );
}
return modified;
}
@Override
public void onDelete( Object entity, Serializable id, Object[] state, String[] propertyNames, Type[] types )
{
if ( audit && entity instanceof Auditable )
deletes.add( (Auditable)entity );
}
@Override
public void postFlush( Iterator iterator ) throws CallbackException
{
if ( inserts.isEmpty() && deletes.isEmpty() && updates.isEmpty() )
return;
Session session = sessionFactory.openSession();
session.setFlushMode( FlushMode.COMMIT );
session.beginTransaction();
try
{
String action = null;
for ( Auditable entity : inserts )
{
action = "Created " + entity.getClass().getSimpleName() + " " + entity.toString();
session.save( new PatientChangeLog( entity.getPatient(), action, entity.getReason(), SessionController.userName ) );
}
for ( Auditable entity : deletes )
{
action = "Deleted " + entity.getClass().getSimpleName() + " " + entity.toString();
session.save( new PatientChangeLog( entity.getPatient(), action, entity.getReason(), SessionController.userName ) );
}
for ( UpdatedEntity entity : updates )
{
Auditable a = entity.getAuditable();
StringBuffer actionBuf = new StringBuffer( "Updated " + a.getClass().getSimpleName() + " " + a.toString() + ": changed " );
int count = 0;
for ( int i = 0; i < entity.getPropertyNames().length; i++ )
{
String prop = entity.getPropertyNames()[i];
Type type = entity.getTypes()[i];
Object curValue = entity.getCurrentState()[i];
Object prevValue = entity.getPreviousState()[i];
//Don't consider the id field or the metadata fields.
if ( prop.equals( "id" ) || prop.equals( "createUser" ) || prop.equals( "createDate" ) || prop.equals( "modifyUser" )
|| prop.equals( "modifyDate" ) || prop.equals( "guid" ) )
continue;
if ( prevValue == null )
prevValue = new String( "" );
if ( curValue == null )
curValue = new String( "" );
if ( !prevValue.equals( curValue ) )
{
if ( count > 0 )
actionBuf.append( " and " );
actionBuf.append( prop ).append( " from '" ).append( prevValue ).append( "' to '" ).append( curValue ).append( "'" );
count++;
}
}
Patient p = (Patient)entity.getPrevious( "patient" ); //In case the patient is changed, tie it to the previous patient.
session.save( new PatientChangeLog( p, actionBuf.toString(), a.getReason(), SessionController.userName ) );
}
session.getTransaction().commit();
}
catch ( HibernateException e )
{
try
{
session.getTransaction().rollback();
}
catch ( Exception hex )
{
throw new RuntimeException( hex );
}
throw new RuntimeException( e );
}
finally
{
inserts.clear();
updates.clear();
deletes.clear();
session.close();
}
}
}
非常感谢任何帮助。
答案 0 :(得分:2)
无法找到合适的文档来代替Envers使用Hibernate事件,我决定将代码从MyInterceptor.postFlush()
移到MyInterceptor.afterTransactionCompletion()
。这样,在我为审计日志记录打开新会话/事务之前,当前事务已提交。感谢您帮助我们做到这一点。
答案 1 :(得分:1)
它显然看起来像事务之间的死锁:审计事务等待主事务锁定的某些资源,但主事务无法提交,因为拦截器的执行被阻止。
虽然Interceptor
的机制不允许,但在同一事务中执行审计操作可能会在概念上更加干净。您可以尝试使用events - 至少,Envers将它们用于审核操作。
答案 2 :(得分:0)
我猜你在审计交易和Chemo交易之间的数据库中有一个死锁。