🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@Linchin

Can you take a look at this and review/approve?

• The new unit test is covered by Kokoro.
• The old sqlalchemy compliance test related to Windowing is deleted/not run.

There used to be two Windowing compliance tests, the second one no longer runs. The test sequence runs the first Windowing test (test_windwo) and not the second:

.../test_dialect_compliance.py::WeCanSetDefaultSchemaWEventsTest::test_wont_work_wo_insert <- .nox/compliance/lib/python3.12/site-packages/sqlalchemy/testing/suite/test_dialect.py SKIPPED [ 99%]
.../test_dialect_compliance.py::WindowFunctionTest_bigquery+bigquery::test_window <- .nox/compliance/lib/python3.12/site-packages/sqlalchemy/testing/suite/test_select.py PASSED [100%]

NOTE: this solves one specific flaky-bot issue related to Windowing. It does not solve the other flakybot issues related to the Enum errors. I want to get this resolution on the books so we can clear out the flakybot issue. I am working in a separate PR on fixes to handle the Enum errors.

Will tag this PR with the PR number for the Enum issue as soon as possible.

Hi @chalmerlowe, thank you for taking the time to fix the flaky issue! Indeed now only the enum tests are failing.

LGTM.