carloea2 opened a new pull request, #8262:
URL: https://github.com/apache/texera/pull/8262

   ### What changes were proposed in this PR?
   
   Treat the S3 missing-bucket ClientError codes from `HeadBucket` as a signal 
to create the large binary bucket. Preserve all other client errors.
   
   The regression covers both a 404 missing bucket and a 403 error that must 
propagate.
   
   ### Any related issues, documentation, discussions?
   
   Closes #8261
   
   ### How was this PR tested?
   
   A botocore S3 stub reproduced the untouched failure with `ClientError: 404`. 
After the fix, it consumed both `HeadBucket` and `CreateBucket` and reported 
`bucket_created=True`.
   
   `C:\Users\carlo\texera\texera\.venv312\Scripts\python.exe -c "import 
sys,pytest; 
sys.path[:0]=[r'C:\Users\carlo\texera\texera-worktrees\investigate-bug67\amber\src\main\python',r'C:\Users\carlo\texera\texera\amber\src\main\python'];
 raise 
SystemExit(pytest.main([r'amber/src/test/python/pytexera/storage/test_large_binary_manager.py','-q','-p','no:cacheprovider']))"`
   
   `C:\Users\carlo\texera\texera\.venv312\Scripts\ruff.exe check 
amber/src/main/python amber/src/test/python`
   
   `C:\Users\carlo\texera\texera\.venv312\Scripts\ruff.exe format --check 
amber/src/main/python/pytexera/storage/large_binary_manager.py 
amber/src/test/python/pytexera/storage/test_large_binary_manager.py`
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: OpenAI Codex


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to