sstableloader: fix handling of rewritten partition
sstableloader: fix handling of rewritten partition

Before this patch, when sstableloader saw in an sstable a partition that
had a partiton tombstone, it generated a deletion command and stopped
processing this partition. But this is wrong - a partition can have a
tombstone *and* data. We need to write both the deletion and the data!

We noticed this bug when loading tables created by Alternator.
Alternator's PutItem operation replaces an item and always writes a
tombstone at time t-1 and data at time t - and sstableloader lost all
this data.

Fixes #238.

Signed-off-by: Nadav Har'El <625169824b719df98706d88646d0aae6f368ae4d@scylladb.com>
Message-Id: <20210401130537.69308-1-625169824b719df98706d88646d0aae6f368ae4d@scylladb.com>

scylladb/scylla-tools-java,scylladb/scylla-tools-java,scylladb/scylla-tools-java,scylladb/scylla-tools-java