Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow partial support for filtering against window functions #239

Merged
merged 1 commit into from
Mar 23, 2023

Conversation

maikhanhbui
Copy link
Contributor

@maikhanhbui maikhanhbui commented Mar 15, 2023

This PR allows partial support for filtering against window functions, which was added in Django 4.2.

Fixes the following Django 4.2 tests:

expressions_window.tests.WindowFunctionTests.test_exclude,
expressions_window.tests.WindowFunctionTests.test_filter,
expressions_window.tests.WindowFunctionTests.test_filter_alias,
expressions_window.tests.WindowFunctionTests.test_filter_column_ref_rhs,
expressions_window.tests.WindowFunctionTests.test_filter_conditional_annotation,
expressions_window.tests.WindowFunctionTests.test_filter_conditional_expression,
expressions_window.tests.WindowFunctionTests.test_filter_count,
expressions_window.tests.WindowFunctionTests.test_filter_select_related,
expressions_window.tests.WindowFunctionTests.test_filter_values,
expressions_window.tests.WindowFunctionTests.test_heterogeneous_filter,

The following Django 4.2 WindowFunction test failure still remains open:

expressions_window.tests.WindowFunctionTests.test_limited_filter

@maikhanhbui maikhanhbui merged commit 5a630ba into django42 Mar 23, 2023
@maikhanhbui maikhanhbui deleted the fix-windows-tests branch March 23, 2023 23:46
mShan0 added a commit that referenced this pull request May 31, 2023
* Allow Django 4.2 (#227)

* Allow Django 4.2

* allow Django 4.2

* Fix errors with raising FullResultSet exception and with alter_column_type_sql() and collate_sql() functions (#229)

* fix error with raising fullresultset

* add django4.2 condition

* fix alter_column_type_sql and collate_sql to take 2 additional arguments

* delete argument 'old_rel_collation'

* fix arguments names

* fix last_executed_query() to properly replace placeholders with params (#234)

* disable allows_group_by_select_index

* unskip old tests

* unskip some tests

* skip more tests

* Use latest Django 4.2 beta for tox (#238)

* use 4.2 rc1 branch (#240)

* allow partial support for filtering against window functions (#239)

* add subsecond support to Now() (#242)

* assign value to display_size (#244)

* add latest django 4.2 branch to ci

* raise an error when batch_size is zero. (#259)

* replicate get or create test for mssql (#265)

* Add skipped tests to Django 4.2 (#268)

* skip django 4.2 failing tests

* skip schema test

* skip aggregate annotation pruning test

---------

Co-authored-by: mShan0 <mark.shan19@gmail.com>

* syntax fix

* ci fix

* bump version to 1.3

---------

Co-authored-by: Khanh Bui <85855766+khanhmaibui@users.noreply.github.com>
Co-authored-by: Khanh Bui <khanhmb815@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants