Luke VanderHart

Results 26 comments of Luke VanderHart

@jeffchuber this PR now includes updated code for the JS client if you wouldn't mind taking a look 🙏 It's ready to merge as soon as it's reviewed and we...

Converting back to draft until I fix some inconsistencies caused by the rebase on the new `main`.

@HammadB Changes we discussed live yesterday are in place, should be ready to go.

@HammadB just bouncing something off you. Most the code up until now presumes either UUIDs or strings for embedding IDs. Managing multiple types across multiple databases was super cumbersome, so...

@HammadB most DBs have a dedicated UUID type or at least fixed-length binary data (which is effectively the same.) Strangely though, CrateDB is not one of them: we'd have to...

> Top level question - how are segment types assigned to collections? Are you imagining it as a data-responsive process in the placement engine - thats what I am assuming,...

Closing because this is too big and too far out of date. Going to re-create using smaller PRs.

We are keeping the API the same and there are also some performance and operational benefits to having as much validation as we can up front. Cost is, as you...

Overriding required check and merging, CI for this branch is currently not working but per discussion we will resolve that as a separate issue.

Reviewed in-person with @atroyn and incorporated feedback prior to merging.