zeroshade commented on PR #280: URL: https://github.com/apache/iceberg-go/pull/280#issuecomment-2631914600
@kevinjqliu No, `rest_test.go` is running tests against mocked endpoints, it's not hitting the iceberg REST image. It looks like this needs to update from the recent merges and be passing a `context.Background()` as the first argument to the `rest.NewCatalog` functions, that's what's causing the errors in the CI. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
