Remove StorageID from committed.iterator

[itaigilo]

Cleanup:
The storageID of committed.manager isn't used,
Removing it.

[guy-har]

LGTM

[nopcoder]

Looks good as it seems that it should never had to have the storage id in the first place.

[github-actions]

E2E Test Results - DynamoDB Local - Local Block Adapter

Test failures:
Object Viewer - Parquet File › view parquet object w/ logout and login: common/viewParquetObject.spec.ts viewParquetObject.spec.ts:28:9 view parquet object w/ logout and login [common] › common/viewParquetObject.spec.ts:28:9 › Object Viewer - Parquet File › view parquet object w/ logout and login Error: page.goto: Navigation to "http://localhost:8000/repositories" is interrupted by another navigation to "http://localhost:8000/" Call log: - navigating to "http://localhost:8000/repositories", waiting until "load" at poms/repositoriesPage.ts:25 23 | 24 | async goto(): Promise<void> { > 25 | await this.page.goto("/repositories"); | ^ 26 | } 27 | 28 | async goToRepository(repoName: string): Promise<void> { at RepositoriesPage.goto (/home/runner/work/lakeFS/lakeFS/webui/test/e2e/poms/repositoriesPage.ts:25:25) at /home/runner/work/lakeFS/lakeFS/webui/test/e2e/common/viewParquetObject.spec.ts:41:32 attachment #1: trace (application/zip) ───────────────────────────────────────────────────────── test-results/common-viewParquetObject-Object-Viewer---Parquet-File-view-parquet-object-w-logout-and-login-common/trace.zip Usage: npx playwright show-trace test-results/common-viewParquetObject-Object-Viewer---Parquet-File-view-parquet-object-w-logout-and-login-common/trace.zip ────────────────────────────────────────────────────────────────────────────────────────────────
This test report was produced by the test-summary action.  Made with ❤️ in Cambridge.

[github-actions]

E2E Test Results - Quickstart