You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@kyuubi.apache.org by "wForget (via GitHub)" <gi...@apache.org> on 2026/08/28 14:19:05 UTC
[PR] [KYUUBI #7674] Fix flaky PyHive SQLAlchemy insert tests [kyuubi]
wForget opened a new pull request, #7675:
URL: https://github.com/apache/kyuubi/pull/7675
### Why are the changes needed?
The Python test suite runs with multiple pytest-xdist workers. `test_insert_select` and `test_insert_values` used the same `pyhive_test_database.insert_test` table, and both tests dropped and recreated it.
When the tests ran concurrently, one test could drop the table while the other was inserting or querying data, causing intermittent `table not found` failures.
This patch assigns a dedicated table name to each test to prevent the collision.
Closes #7674.
### How was this patch tested?
Existing test cases.
### Was this patch assisted by generative AI tooling?
Assisted-by: OpenAI Codex (GPT-5)
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
Re: [PR] [KYUUBI #7674] Fix flaky PyHive SQLAlchemy insert tests [kyuubi]
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 closed pull request #7675: [KYUUBI #7674] Fix flaky PyHive SQLAlchemy insert tests
URL: https://github.com/apache/kyuubi/pull/7675
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org
Re: [PR] [KYUUBI #7674] Fix flaky PyHive SQLAlchemy insert tests [kyuubi]
Posted by "pan3793 (via GitHub)" <gi...@apache.org>.
pan3793 commented on PR #7675:
URL: https://github.com/apache/kyuubi/pull/7675#issuecomment-5476411687
**Merge Summary:**
- merged into master https://github.com/apache/kyuubi/commit/044f8816db4a61c454c584ad4ba83bed0695f761
- merged into branch-1.12 https://github.com/apache/kyuubi/commit/9fd49a03b9a4054bfaa2c50fa4ffb9f5ea9d5845
- merged into branch-1.11 https://github.com/apache/kyuubi/commit/fa649f61c3d2cc74a194a633020248c5684aa415
- merged into branch-1.10 https://github.com/apache/kyuubi/commit/13c85c1c141abf58a1b4e767700bc20d0c0c9af0
*Posted by `merge_kyuubi_pr.py`*
--
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: notifications-unsubscribe@kyuubi.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@kyuubi.apache.org
For additional commands, e-mail: notifications-help@kyuubi.apache.org