Ypatia Tsavliri

> @ypatia is there anything else to do on this PR? If not can we approve it? It's ok for me, but let's have an approval from either @Shelnutt2 or...

As per @Shelnutt2's initial review we have: - Removed `flush()` - Removed the config variable all along as it was only needed to disable `flush()` if that would bring perf...

> I'll get to the review on this today, but we should probably flip this to draft so we don't merge it until after the cloud team has merged the...

> Lets make this a draft to avoid an accidental merge. > > We should also add tests for the REST CI runner. Those are conditionally enabled here: > >...

> LGTM once CI is green, nice work 👍 REST CI will fail since this PR removes bootstrap flags being used in those workflows. > > If you want to...

This work has been covered by this https://github.com/TileDB-Inc/TileDB-Go/pull/329 , closing as not needed anymore.

FYI, the REST-CI failure should go away if you rebase to latest `dev` , it's due to [sc-44928](https://app.shortcut.com/tiledb-inc/story/44928)

@KiterLuc this is a fix in `consolidate_fragments` needed for shipping Fragment list consolidation of `tiledb://` arrays. Do you think we can get this in the upcoming 2.24.1 patch release? Thanks.

> > @KiterLuc this is a fix in `consolidate_fragments` needed for shipping Fragment list consolidation of `tiledb://` arrays. Do you think we can get this in the upcoming 2.24.1 patch...

@KiterLuc we should discuss before merging this in case we first need to address this one first: https://app.shortcut.com/tiledb-inc/story/52302/increase-number-of-max-fragments-in-tiledb-fragment-list-consolidation