The GitHub Actions job "Required Checks" on 
texera.git/gh-readonly-queue/main/pr-8264-74c23f1f317b867db89ca96118485fd5628990da
 has failed.
Run started by GitHub user Yicong-Huang (triggered by Yicong-Huang).

Head commit for run:
64818bc0012fd779875c3e9f6861a1cf1d9a6ab0 / carloea2 <[email protected]>
feat(pyamber): resolve unset input port zero (#8264)

### What changes were proposed in this PR?

Normalize unset protobuf port fields before the dictionary lookup,
keeping lookup O(1). This lets an unset port-zero identity find the
registered port and preserves a clear KeyError for unknown ports.

### Any related issues, documentation, discussions?

Closes #8263

### How was this PR tested?

The untouched live probe showed equal port-zero identities with
different hashes and raised `KeyError`. After the fix it reported
`lookup_succeeded=True` with the hashes still different.

`C:\Users\carlo\texera\texera\.venv312\Scripts\python.exe -c "import
sys,pytest;
sys.path[:0]=[r'C:\Users\carlo\texera\texera-worktrees\investigate-bug68\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_input_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/core/architecture/packaging/input_manager.py
amber/src/test/python/core/architecture/packaging/test_input_manager.py`

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

Generated-by: Codex

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

With regards,
GitHub Actions via GitBox

Reply via email to