From 28e14a08d876927f255cf3ebd4ed66913ec74607 Mon Sep 17 00:00:00 2001 From: "Raphael S. Carvalho" Date: Thu, 21 May 2015 15:09:46 -0300 Subject: [PATCH] sstables: add FIXME comment Signed-off-by: Raphael S. Carvalho --- sstables/sstables.cc | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sstables/sstables.cc b/sstables/sstables.cc index c6b181fb32..6ff3a41905 100644 --- a/sstables/sstables.cc +++ b/sstables/sstables.cc @@ -519,6 +519,9 @@ future<> parse(random_access_reader& in, summary& s) { future<> write(file_writer& out, summary_entry& entry) { + // FIXME: summary entry is supposedly written in memory order, but that + // would prevent portability of summary file between machines of different + // endianness. We can treat it as little endian to preserve portability. return write(out, entry.key).then([&out, &entry] { auto p = reinterpret_cast(&entry.position); return out.write(p, sizeof(uint64_t));