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

   ### What is this PR for?
   
   Remove a channel from its previous input port before registering it with 
another port. This keeps the forward mapping and each port's channel set 
consistent for port-aligned control messages.
   
   Closes #8227
   
   ### What type of PR is it?
   
   Bug Fix
   
   ### How was this PR tested?
   
   ```text
   python -c "import sys,pytest; sys.path[:0]=[worktree amber source, main 
amber source]; raise 
SystemExit(pytest.main(['amber/src/test/python/core/architecture/packaging/test_input_manager.py','amber/src/test/python/core/architecture/managers/test_embedded_control_message_manager.py','amber/src/test/python/core/architecture/handlers/control/test_add_input_channel_handler.py','-q','-p','no:cacheprovider']))"
   40 passed
   
   ruff check amber/src/main/python amber/src/test/python
   All checks passed
   
   ruff format --check amber/src/main/python amber/src/test/python
   213 files already formatted
   ```
   
   A direct post-fix probe confirmed both mapping directions agree:
   
   ```text
   mapped_port=1
   old_port_contains_channel=False
   new_port_contains_channel=True
   old_port_channel_count=0
   ```
   
   Generated-by: 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