Remove friend declaration in datastore
Make DataStore a friend of Transaction, make default ctor of Transaction pvt

Because the default ctor should only be used by DataStore and noone else
Fix lint error, whitespace around &&