TemporaryRecordStore is semantically incorrect for replicated tables, as dropping the table needs to be part of a replicated operation, not just when the c++ object happens to leave scope. For hybrid index builds the tables are unreplicated, but primary-driven index builds replicate them. We can probably make this change for both without issues as the cleanup should happen as part of index build commit or abort.