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.

Sign up for GitHub

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

Jump to bottom

GEOSEARCH & GEOSEARCHSTORE #2089

Merged
merged 10 commits into from
Apr 16, 2022
Merged

GEOSEARCH & GEOSEARCHSTORE #2089

merged 10 commits into from
Apr 16, 2022

Commits on Apr 13, 2022

  1. geosearch feature

    @slorello89
    slorello89 committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    838ccd0 View commit details
    Browse the repository at this point in the history

  2. removing accidental whitespace addition.

    @slorello89
    slorello89 committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    2e202f5 View commit details
    Browse the repository at this point in the history

  3. fixing literals order

    @slorello89
    slorello89 committed Apr 13, 2022
    Configuration menu
    Copy the full SHA
    4595921 View commit details
    Browse the repository at this point in the history

Commits on Apr 14, 2022

  1. adding Argument Exception test-case

    @slorello89
    slorello89 committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    801a2ac View commit details
    Browse the repository at this point in the history

  2. Cleanup

    @NickCraver
    NickCraver committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    95c1dc5 View commit details
    Browse the repository at this point in the history

  3. Simplify maintenance vs. performance tradeoff 

    This matches GeoRadius today and while not as efficient, is much simpler to maintain in this high-args count case. We can always revisit, but relative to the cost of a search I'd opt for simpler code in this case.
    @NickCraver
    NickCraver committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    98aa54b View commit details
    Browse the repository at this point in the history

  4. Rename to overloads (like GeoRadius) and update docs

    @NickCraver
    NickCraver committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    bba7f8d View commit details
    Browse the repository at this point in the history

  5. Merge remote-tracking branch 'origin/main' into feature/GeoSearch

    @NickCraver
    NickCraver committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    4a63086 View commit details
    Browse the repository at this point in the history

  6. Unify and cleanup tests

    @NickCraver
    NickCraver committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    0c05d4b View commit details
    Browse the repository at this point in the history

  7. Doh, missed one

    @NickCraver
    NickCraver committed Apr 14, 2022
    Configuration menu
    Copy the full SHA
    aae4fb1 View commit details
    Browse the repository at this point in the history