The GitHub Actions job "CI" on iceberg-rust.git/main has succeeded.
Run started by GitHub user blackmwk (triggered by blackmwk).

Head commit for run:
c56411a31945591dc600b95206d956ce0a7bb85c / Brodie Alexander 
<[email protected]>
fix: SqlCatalogBuilder does not save supplied name in SqlCatalogConfig before 
initializing SqlCatalog (#2079)

There hasn't been any movement on #1991 for a while so I wanted to see
if I could help out. I'm not sure what the correct way of trying to
continue where another PR left off is, so please let me know if there is
a better way of going about this!

## Which issue does this PR close?

Closes #1990
Refer to PR #1991

## What changes are included in this PR?

Changes to SqlCatalogBuilder from #1991 are included but the
new_sql_catalog function has been refactored to accept an optional name
parameter. Relevant tests have been adjusted accordingly.

## Are these changes tested?

Yes, refer to #1991 for cornerwings' added test. I've run the local
`make test` and everything seems to check out.

Thanks!
    Brodie

---------

Co-authored-by: Sainath Mallidi <[email protected]>
Co-authored-by: blackmwk <[email protected]>

Report URL: https://github.com/apache/iceberg-rust/actions/runs/21500303096

With regards,
GitHub Actions via GitBox

Reply via email to