Skip to content

Conversation

Fokko
Copy link
Contributor

@Fokko Fokko commented May 27, 2025

Rationale for this change

Are these changes tested?

Are there any user-facing changes?

Copy link
Contributor

@kevinjqliu kevinjqliu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

mc config is deprecated minio/mc#5206

@kevinjqliu kevinjqliu merged commit f7767e0 into apache:main May 27, 2025
10 checks passed
@kevinjqliu
Copy link
Contributor

rebased dependabot PRs

kevinjqliu pushed a commit to apache/iceberg-go that referenced this pull request May 28, 2025
taken from apache/iceberg-python#2049

Looks like `mc config` is deprecated
minio/mc#5206
@Fokko Fokko mentioned this pull request Jun 1, 2025
deepyaman added a commit to dagster-io/community-integrations that referenced this pull request Jun 2, 2025
## Summary & Motivation

- Pinned Spark<4 (but upgraded to latest Spark 3 and related deps)
- Fixed `mc` command, as in apache/iceberg-python#2049
- aio-libs/aiohttp#11094 fixed aio-libs/aiohttp#11091, so the `aiohttp` pin is no longer necessary

## How I Tested These Changes

- CI
liurenjie1024 pushed a commit to apache/iceberg-rust that referenced this pull request Jun 7, 2025
## Which issue does this PR close?

- Closes #.

## What changes are included in this PR?

`mc config` is deprecated minio/mc#5206

Similar to fix in iceberg-python and iceberg-go
apache/iceberg-python#2049
apache/iceberg-go#444

This PR also updates the image version of `minio:minio` and `minio:mc`
to the latest available tag to pull in the change where `mc config` is
deprecated.


## Are these changes tested?
amitgilad3 pushed a commit to amitgilad3/iceberg-python that referenced this pull request Jul 7, 2025
<!--
Thanks for opening a pull request!
-->

<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->

# Rationale for this change

# Are these changes tested?

# Are there any user-facing changes?

<!-- In the case of user-facing changes, please add the changelog label.
-->
gabeiglio pushed a commit to Netflix/iceberg-python that referenced this pull request Aug 13, 2025
<!--
Thanks for opening a pull request!
-->

<!-- In the case this PR will resolve an issue, please replace
${GITHUB_ISSUE_ID} below with the actual Github issue id. -->
<!-- Closes #${GITHUB_ISSUE_ID} -->

# Rationale for this change

# Are these changes tested?

# Are there any user-facing changes?

<!-- In the case of user-facing changes, please add the changelog label.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants