Bug fixes related to txn nil callbacks and badgerMove key.

iterator.go

@@ -167,7 +167,9 @@ func (item *Item) yieldItemValue() ([]byte, func(), error) {
 		// The value pointer is pointing to a deleted value log. Look for the
 		// move key and read that instead.
 		runCallback(cb)
-		key = append(badgerMove, y.KeyWithTs(item.Key(), item.Version())...)
+		// Do not put badgerMove on the left in append. It seems to cause some sort of manipulation.
+		key = append([]byte{}, badgerMove...)
+		key = append(key, y.KeyWithTs(item.Key(), item.Version())...)
 		// Note that we can't set item.key to move key, because that would
 		// change the key user sees before and after this call. Also, this move
 		// logic is internal logic and should not impact the external behavior

transaction.go

@@ -411,7 +411,7 @@ func (txn *Txn) runCallbacks() {
 	for _, cb := range txn.callbacks {
 		cb()
 	}
-	txn.callbacks = nil
+	txn.callbacks = txn.callbacks[:0]
 }
 
 // Discard discards a created transaction. This method is very important and must be called. Commit

value.go

@@ -387,7 +387,8 @@ func (vlog *valueLog) rewrite(f *logFile, tr trace.Trace) error {
 			if bytes.HasPrefix(e.Key, badgerMove) {
 				ne.Key = append([]byte{}, e.Key...)
 			} else {
-				ne.Key = append(badgerMove, e.Key...)
+				ne.Key = append([]byte{}, badgerMove...)
+				ne.Key = append(ne.Key, e.Key...)
 			}
 
 			ne.Value = append([]byte{}, e.Value...)