The GitHub Actions job "Backport Approval Check" on 
texera.git/gh-readonly-queue/main/pr-8278-adc6a4ac4d79f0302534228d119ea2c22f393db7
 has succeeded.
Run started by GitHub user aglinxinyuan (triggered by aglinxinyuan).

Head commit for run:
27bc3b7703c7e044c03a8d76edeb708146441fce / carloea2 <[email protected]>
feat(pyamber): skip duplicate output writers (#8278)

### What changes were proposed in this PR?

Start output storage writers only when the output port is first
registered. Duplicate registration keeps the existing port and writer
set.

### Any related issues, documentation, discussions?

Closes #8277

### How was this PR tested?

Added coverage proving that duplicate port registration does not start
replacement writers, alongside the existing positive writer-setup
coverage.

`C:\Users\carlo\texera\texera\.venv312\Scripts\python.exe -c "import
sys,pytest;
sys.path[:0]=[r'C:\Users\carlo\texera\texera-worktrees\investigate-bug74\amber\src\main\python',r'C:\Users\carlo\texera\texera\amber\src\main\python'];
raise
SystemExit(pytest.main([r'amber\src\test\python\core\architecture\packaging\test_output_manager.py','-p','no:cacheprovider','-q']))"`

`C:\Users\carlo\texera\texera\.venv312\Scripts\ruff.exe check
amber/src/main/python/core/architecture/packaging/output_manager.py
amber/src/test/python/core/architecture/packaging/test_output_manager.py`

`C:\Users\carlo\texera\texera\.venv312\Scripts\ruff.exe format --check
amber/src/main/python/core/architecture/packaging/output_manager.py
amber/src/test/python/core/architecture/packaging/test_output_manager.py`

All 40 tests passed. Ruff checks passed.

### Was this PR authored or co-authored using generative AI tooling?

Generated-by: Codex

Report URL: https://github.com/apache/texera/actions/runs/38183365063

With regards,
GitHub Actions via GitBox

Reply via email to