Closed tychoish closed 1 day ago
In talking with @greyscaled, it became clear to us that we can drop all of the new configuration except the bucket name from the configuration options, because the access to that bucket will be added to the "main" credentials, which makes a lot of the code more simple.
I want to hold of merging this until after things calm down a bit and we have time to coordinate landing this with cloud.
adding cloud label, with love ❤️
Want to run some things through CI.
We sholdn't merge this as it'll require some changes to cloud (granting access to the appropriate nodes,) passing new arguments to the service, which we should do some testing to figure out how to do that first.
Once this does land we can remove all the client/service abstraction and clean up a fair amount of the code, which might make some things more clear (and make the metastore just a storage subsystem, and interace onto the catalog).
Closes #560