Skip to content

[read-unfinalized-object] [part2] Enable read of unfinalized objects during streaming writes and the corresponding disabled e2e tests #3405

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 10 commits into
base: gargnitin/support-unfinalized-read/read-cached/v1
Choose a base branch
from

Conversation

gargnitingoogle
Copy link
Collaborator

@gargnitingoogle gargnitingoogle commented Jun 11, 2025

Description

Link to the issue in case of a bug fix.

b/425828097
b/410698332
b/422355580

Testing details

  1. Manual - NA
  2. Unit tests - NA
  3. Integration tests - Ran (log) as part of presubmit .

Any backward incompatible change? If so, please explain.

@gargnitingoogle gargnitingoogle changed the base branch from master to gargnitin/support-unfinalized-read/read-cached/v1 June 11, 2025 11:59
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 9deb5ea to 170a49a Compare June 11, 2025 18:08
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from d3506fa to 96a595e Compare June 11, 2025 18:08
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 170a49a to 4af9051 Compare June 12, 2025 10:57
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 96a595e to c156f74 Compare June 12, 2025 10:57
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 4af9051 to d20e570 Compare June 12, 2025 11:02
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from c156f74 to d747822 Compare June 12, 2025 11:02
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from d20e570 to a5c094a Compare June 13, 2025 04:50
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from d747822 to 40a2fe6 Compare June 13, 2025 04:50
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from a5c094a to f7ace70 Compare June 13, 2025 06:45
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 40a2fe6 to ca48921 Compare June 13, 2025 06:45
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from f7ace70 to 04130dd Compare June 16, 2025 06:01
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from ca48921 to 38c87c7 Compare June 16, 2025 06:01
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 04130dd to 9568b19 Compare June 16, 2025 12:36
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 38c87c7 to 2fe84ab Compare June 16, 2025 12:36
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 9568b19 to 3869b40 Compare June 17, 2025 05:38
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 2fe84ab to 942200a Compare June 17, 2025 05:39
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 3869b40 to ba62f12 Compare June 17, 2025 11:15
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch 7 times, most recently from 7b28fd6 to 22b85ca Compare June 18, 2025 07:37
@gargnitingoogle gargnitingoogle changed the title [read-unfinalized-object] Always use latest size [read-unfinalized-object] Enable first e2e test for ZB unfinalized object read Jun 18, 2025
@gargnitingoogle gargnitingoogle added the execute-integration-tests-on-zb To run E2E tests on zonal bucket. label Jun 18, 2025
@gargnitingoogle gargnitingoogle marked this pull request as ready for review June 18, 2025 07:41
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 02e7ef7 to 063a978 Compare June 24, 2025 08:37
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 3e2759e to 563202a Compare June 25, 2025 04:15
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 063a978 to dc44799 Compare June 25, 2025 04:15
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 563202a to ffac771 Compare June 25, 2025 11:08
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch 2 times, most recently from 4c4f5a8 to 70f061a Compare June 25, 2025 11:47
@gargnitingoogle gargnitingoogle added the execute-integration-tests-on-zb To run E2E tests on zonal bucket. label Jun 25, 2025
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 70f061a to 216db8f Compare June 25, 2025 12:50
@gargnitingoogle gargnitingoogle changed the base branch from gargnitin/support-unfinalized-read/read-cached/v1 to master June 25, 2025 12:51
@gargnitingoogle gargnitingoogle removed the execute-integration-tests-on-zb To run E2E tests on zonal bucket. label Jun 25, 2025
@gargnitingoogle gargnitingoogle changed the base branch from master to gargnitin/support-unfinalized-read/read-cached/v1 June 25, 2025 12:56
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v1 branch from 1598bd3 to a45cebb Compare June 26, 2025 06:33
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 6a107ba to 399c958 Compare June 26, 2025 06:33
@gargnitingoogle gargnitingoogle changed the base branch from gargnitin/support-unfinalized-read/read-cached/v1 to master June 26, 2025 08:16
@gargnitingoogle gargnitingoogle added the execute-integration-tests-on-zb To run E2E tests on zonal bucket. label Jun 26, 2025
@gargnitingoogle gargnitingoogle force-pushed the gargnitin/support-unfinalized-read/read-cached/v2 branch from 399c958 to 8744f9a Compare June 26, 2025 08:16
@gargnitingoogle gargnitingoogle removed the execute-integration-tests-on-zb To run E2E tests on zonal bucket. label Jun 26, 2025
@gargnitingoogle gargnitingoogle changed the base branch from master to gargnitin/support-unfinalized-read/read-cached/v1 June 26, 2025 08:21
@gargnitingoogle gargnitingoogle added the remind-reviewers Auto remind reviewers in attention set for review post 24hrs of inactivity on PR. label Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
remind-reviewers Auto remind reviewers in attention set for review post 24hrs of inactivity on PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants