Skip to content

fix(deps, pyamber): bump s3fs from 2025.9.0 to 2026.6.0 in /amber#6232

Open
dependabot[bot] wants to merge 4 commits into
mainfrom
dependabot/pip/amber/s3fs-2026.6.0
Open

fix(deps, pyamber): bump s3fs from 2025.9.0 to 2026.6.0 in /amber#6232
dependabot[bot] wants to merge 4 commits into
mainfrom
dependabot/pip/amber/s3fs-2026.6.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 7, 2026

Copy link
Copy Markdown
Contributor

Bumps s3fs from 2025.9.0 to 2026.6.0.

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [s3fs](https://github.com/fsspec/s3fs) from 2025.9.0 to 2026.6.0.
- [Changelog](https://github.com/fsspec/s3fs/blob/main/release-procedure.md)
- [Commits](https://github.com/fsspec/s3fs/commits)

---
updated-dependencies:
- dependency-name: s3fs
  dependency-version: 2026.6.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file python Pull requests that update python code labels Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Automated Reviewer Suggestions

Based on the git blame history of the changed files, we recommend the following reviewers:

  • No candidates found from git blame history.

@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

⚠️ Benchmark changes need a look

🟢 4 better · 🔴 3 worse · ⚪ 8 noise (<±5%) · 0 without baseline

Compared against main 27f145a benchmarked on this same runner, so the delta is largely free of cross-runner hardware noise. The "7d avg" column still reflects the gh-pages dashboard. Treat <±5% as noise unless repeated.

Dashboard · Run

config throughput MB/s latency max Δ latest / 7d
🔴 bs=10 sw=10 sl=64 409 0.25 24,917/28,674/28,674 us 🟢 -20.7% / 🔴 +93.6%
🟢 bs=100 sw=10 sl=64 832 0.508 119,017/133,001/133,001 us 🟢 -16.5% / 🔴 +21.9%
🔴 bs=1000 sw=10 sl=64 911 0.556 1,096,343/1,175,067/1,175,067 us 🔴 +7.2% / 🔴 +12.3%
Baseline details

Latest main 27f145a from same runner

config metric PR latest main 7d avg Δ latest Δ 7d
bs=10 sw=10 sl=64 throughput 409 tuples/sec 426 tuples/sec 760.25 tuples/sec -4.0% -46.2%
bs=10 sw=10 sl=64 MB/s 0.25 MB/s 0.26 MB/s 0.464 MB/s -3.8% -46.1%
bs=10 sw=10 sl=64 p50 24,917 us 23,240 us 12,873 us +7.2% +93.6%
bs=10 sw=10 sl=64 p95 28,674 us 36,145 us 15,648 us -20.7% +83.2%
bs=10 sw=10 sl=64 p99 28,674 us 36,145 us 19,338 us -20.7% +48.3%
bs=100 sw=10 sl=64 throughput 832 tuples/sec 827 tuples/sec 974.75 tuples/sec +0.6% -14.6%
bs=100 sw=10 sl=64 MB/s 0.508 MB/s 0.505 MB/s 0.595 MB/s +0.6% -14.6%
bs=100 sw=10 sl=64 p50 119,017 us 118,037 us 102,470 us +0.8% +16.1%
bs=100 sw=10 sl=64 p95 133,001 us 159,241 us 109,144 us -16.5% +21.9%
bs=100 sw=10 sl=64 p99 133,001 us 159,241 us 115,530 us -16.5% +15.1%
bs=1000 sw=10 sl=64 throughput 911 tuples/sec 938 tuples/sec 1,000 tuples/sec -2.9% -8.9%
bs=1000 sw=10 sl=64 MB/s 0.556 MB/s 0.572 MB/s 0.61 MB/s -2.8% -8.9%
bs=1000 sw=10 sl=64 p50 1,096,343 us 1,069,741 us 1,005,545 us +2.5% +9.0%
bs=1000 sw=10 sl=64 p95 1,175,067 us 1,095,681 us 1,045,968 us +7.2% +12.3%
bs=1000 sw=10 sl=64 p99 1,175,067 us 1,095,681 us 1,076,800 us +7.2% +9.1%
Raw CSV
config_idx,batch_size,schema_width,string_len,num_batches,total_ms,total_tuples,total_bytes,tuples_per_sec,mb_per_sec,lat_p50_us,lat_p95_us,lat_p99_us
0,10,10,64,20,488.52,200,128000,409,0.250,24917.05,28673.53,28673.53
1,100,10,64,20,2402.77,2000,1280000,832,0.508,119016.66,133000.92,133000.92
2,1000,10,64,20,21964.95,20000,12800000,911,0.556,1096343.28,1175066.90,1175066.90

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Amber Python dependency set by bumping the pinned s3fs version used (transitively) for PyIceberg’s fsspec-based S3 FileIO integration.

Changes:

  • Bump s3fs from 2025.9.0 to 2026.6.0 in amber/requirements.txt.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov-commenter

codecov-commenter commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.43%. Comparing base (27f145a) to head (f6c9814).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #6232      +/-   ##
============================================
- Coverage     60.74%   60.43%   -0.32%     
+ Complexity     3368     3352      -16     
============================================
  Files          1135     1133       -2     
  Lines         44188    43913     -275     
  Branches       4830     4774      -56     
============================================
- Hits          26844    26537     -307     
- Misses        15889    15922      +33     
+ Partials       1455     1454       -1     
Flag Coverage Δ *Carryforward flag
access-control-service 70.00% <ø> (ø) Carriedforward from 4c3d2ba
agent-service 44.59% <ø> (ø) Carriedforward from 4c3d2ba
amber 65.98% <ø> (-0.17%) ⬇️ Carriedforward from 4c3d2ba
computing-unit-managing-service 0.00% <ø> (ø) Carriedforward from 4c3d2ba
config-service 52.30% <ø> (ø) Carriedforward from 4c3d2ba
file-service 63.97% <ø> (ø) Carriedforward from 4c3d2ba
frontend 51.93% <ø> (-0.72%) ⬇️ Carriedforward from 4c3d2ba
notebook-migration-service 78.57% <ø> (ø) Carriedforward from 4c3d2ba
pyamber 91.13% <ø> (-0.05%) ⬇️
workflow-compiling-service 55.14% <ø> (ø) Carriedforward from 4c3d2ba

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file pyamber python Pull requests that update python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants