search

trinodb / aws-proxy

Proxy for S3
Apache License 2.0
8 stars 5 forks source link

Rename and move TrinoS3ProxyConfig to TrinoAwsProxyConfig #114

Closed Randgalt closed 3 months ago

Randgalt commented 3 months ago

Better reflect its usage

Note: no code changes. Just renaming/moving.

Randgalt commented 3 months ago

Do we want to add a test for the config class?

I thought they did this in Trino because of the possibility of Connectors/Plugins loading based on old Config definitions. I think that's not an issue for this project. We don't test configs in Starburst Galaxy. So, I'd say it's not needed. But we can do it if folks want to.