Skip to content

Docs improvements #132

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

Merged
merged 3 commits into from
Feb 7, 2021
Merged

Docs improvements #132

merged 3 commits into from
Feb 7, 2021

Conversation

nils-braun
Copy link
Collaborator

No description provided.

@nils-braun nils-braun mentioned this pull request Feb 7, 2021
4 tasks
@nils-braun nils-braun merged commit 7273c2d into main Feb 7, 2021
@nils-braun nils-braun deleted the feature/docs-improvements branch February 7, 2021 14:34
demianw added a commit to NeuroLang/dask-sql that referenced this pull request Feb 8, 2021
commit e5fac1a
Author: Nils Braun <nils-braun@users.noreply.github.com>
Date:   Sun Feb 7 16:20:55 2021 +0100

    Aggregate improvements and SQL compatibility (dask-contrib#134)

    * A lot of refactoring the the groupby. Mainly to include both distinct and null-grouping

    * Test for non-dask aggregations

    * All NaN data needs to go into the same partition (otherwise we can not sort)

    * Fix compatibility with SQL on null-joins

    * Distinct is not needed, as it is optimized away from Calcite

    * Implement is not distinct

    * Describe new limitations and remove old ones

    * Added compatibility test from fugue

    * Added a test for sorting with multiple partitions and NaNs

    * Stylefix

commit 7273c2d
Author: Nils Braun <nils-braun@users.noreply.github.com>
Date:   Sun Feb 7 15:34:55 2021 +0100

    Docs improvements (dask-contrib#132)

    * Fixed a bug in function references in docs

    * More details on the dask-sql internals

commit bdc518e
Author: Nils Braun <nils-braun@users.noreply.github.com>
Date:   Sun Feb 7 14:19:50 2021 +0100

    Fix the fugue dependency (dask-contrib#133)
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.

1 participant