+1 to new flags also from me
On 26/7/22 18:39, Andrés de la Peña wrote:
I think that's right, using a closed range makes sense to consume the
data provided by "sstablemetadata", which also provides closed ranges.
Especially because with half-open ranges we couldn't compact a sstable
with a sin
I think that's right, using a closed range makes sense to consume the data
provided by "sstablemetadata", which also provides closed ranges.
Especially because with half-open ranges we couldn't compact a sstable with
a single big partition, of which we might only know the token but no the
partition
Reading the responses here and taking a step back, I think the current behavior
of nodetool compact is probably the correct behavior. The main use case I can
see for using nodetool compact is someone wants to take some sstable and
compact it with all the overlapping sstables. So you run “sstab
+1 to new flags. A released, albeit undocumented, behavior is still a
contract with the end user. Flags (and documentation) seem like the right
path to address the situation.
Cheers,
Derek
On Tue, Jul 26, 2022 at 7:28 AM Benedict Elliott Smith
wrote:
>
> I think a change like this could be dan
+1 to Benedict and Ekaterina's points - adding new flags to explicitly
introduce the new behavior and documenting the hell out of both the default and
the new flags seems like the right play. There's quite possibly a lot of
tooling out there in the wild that relies on the current behavior and br
I also like the idea with the flags plus improving the documentation as my
understanding was that it is not really documented and can be confusing and
risky for end users.
On Tue, 26 Jul 2022 at 9:28, Benedict Elliott Smith
wrote:
>
> I think a change like this could be dangerous for a lot of ex
I think a change like this could be dangerous for a lot of existing automation
built atop nodetool.
I’m not sure this change is worthwhile. I think it would be better to introduce
e.g. -ste and -ete for “start token exclusive” and “end token exclusive” so
that users can opt-in to whichever sc
+1, I think that makes the most sense.
Kind Regards,
Brandon
On Tue, Jul 26, 2022 at 8:19 AM J. D. Jordan wrote:
>
> I like the third option, especially if it makes it consistent with repair,
> which has supported ranges longer and I would guess most people would think
> the compact ranges wor
I like the third option, especially if it makes it consistent with repair,
which has supported ranges longer and I would guess most people would think the
compact ranges work the same as the repair ranges.
-Jeremiah Jordan
> On Jul 26, 2022, at 6:49 AM, Andrés de la Peña wrote:
>
>
> Hi all
Hi all,
CASSANDRA-17575 has detected that token ranges in nodetool compact are
interpreted as closed on both sides. For example, the command "nodetool
compact -st 10 -et 50" will compact the tokens in [10, 50]. This way of
interpreting token ranges is unusual since token ranges are usually
half-op
10 matches
Mail list logo
