DataStore: don't call close() in dtor, assert on user calling it

Otherwise we have to call a virtual function from the destructor, which
may have undesirable side-effects.

Found by clang-tidy