TxnLocal.isInitialized -- why doesn't an InTxnEnd suffice?

[Hanns Holger Rutz]

I'm just refactoring some stuff involving `TxnLocal` and `setExternalDecider`. I'm using the `TxnLocal` to store/initialize a peer database transaction. The external decider is now exterior and cascades, i.e. calls into the database decider; that cannot determine whether the `TxnLocal` was initialized, as I only have an `InTxnEnd`; while I can call `apply`, I need to call `isInitialized` first in order to only retrieve the peer txn _if it had been used in that transaction_.

So I wonder why `isInitialized` needs a full `InTxn` and not just `InTxnEnd`, and whether I can work around this somehow?

Thanks, .h.h.

[Hanns Holger Rutz]

the work around

private val dbTxnInit = TxnLocal( false )
private val dbTxnRef = TxnLocal( initialValue = { implicit tx =>
// Txn.setExternalDecider( this )
val res = database.beginTransaction()
...
dbTxnInit() = true
res
}

def shouldCommit( implicit txn: InTxnEnd ) : Boolean = {
// if( !dbTxnRef.isInitialized ) return true // can't call
if( !dbTxnInit.get ) return true // work around

val dbTxn = dbTxnRef() // safe to read now
...

> --
>
>
>

[Nathan Bronson]

You're right, the parameter is more specific than necessary. I've published 0.8-SNAPSHOT with the fix, and I will update the docs once github becomes writable again. Your workaround is correct.

 - Nathan

--

--
Nathan Grasso Bronson
ngbr...@gmail.com