Skip to content

Support GEOSHAPE field type in RediSearch#3561

Merged
sazzad16 merged 7 commits into
redis:masterfrom
sazzad16:ft-geoshape
Oct 11, 2023
Merged

Support GEOSHAPE field type in RediSearch#3561
sazzad16 merged 7 commits into
redis:masterfrom
sazzad16:ft-geoshape

Conversation

@sazzad16

Copy link
Copy Markdown
Contributor

No description provided.

@sazzad16 sazzad16 added this to the 5.1.0 milestone Sep 26, 2023
@sazzad16 sazzad16 requested review from chayim and uglide September 26, 2023 11:15
@sazzad16

Copy link
Copy Markdown
Contributor Author

@chayim @uglide 🔔

@bsbodden 👁️

@sazzad16 sazzad16 marked this pull request as draft September 26, 2023 17:18
GeoShape query is limited to PARAM argument only. So I couldn't implement helper functions in query builders.
Because of that the dependency library is not required.
@codecov-commenter

codecov-commenter commented Sep 27, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (db5bbcd) 71.48% compared to head (a062311) 71.45%.

❗ Current head a062311 differs from pull request most recent head fc57692. Consider uploading reports for the commit fc57692 to get more accurate results

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #3561      +/-   ##
============================================
- Coverage     71.48%   71.45%   -0.04%     
- Complexity     4841     4842       +1     
============================================
  Files           287      288       +1     
  Lines         15432    15446      +14     
  Branches       1093     1093              
============================================
+ Hits          11032    11037       +5     
- Misses         3924     3931       +7     
- Partials        476      478       +2     
Files Coverage Δ
...ava/redis/clients/jedis/search/SearchProtocol.java 97.77% <100.00%> (ø)
...ients/jedis/search/schemafields/GeoShapeField.java 64.28% <64.28%> (ø)

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@sazzad16 sazzad16 marked this pull request as ready for review September 27, 2023 15:21
Comment thread pom.xml
Comment thread src/test/java/redis/clients/jedis/examples/GeoShapeFieldsUsageInRediSearch.java Outdated
@chayim

chayim commented Oct 9, 2023

Copy link
Copy Markdown
Contributor

@uglide IMHO docs... Who can help?

skip-checks: true
@sazzad16 sazzad16 requested a review from chayim October 9, 2023 09:54
@sazzad16 sazzad16 merged commit 5cd40fc into redis:master Oct 11, 2023
@sazzad16 sazzad16 deleted the ft-geoshape branch October 11, 2023 09:33
sazzad16 added a commit that referenced this pull request Oct 18, 2023
* Support GEOSHAPE field type in RediSearch

* dependency:

GeoShape query is limited to PARAM argument only. So I couldn't implement helper functions in query builders.
Because of that the dependency library is not required.

* example

* more comments

* format

* Address code review
sazzad16 added a commit that referenced this pull request Oct 18, 2023
* Support GEOSHAPE field type in RediSearch

* dependency:

GeoShape query is limited to PARAM argument only. So I couldn't implement helper functions in query builders.
Because of that the dependency library is not required.

* example

* more comments

* format

* Address code review
@sazzad16 sazzad16 removed this from the 5.1.0 milestone Oct 18, 2023
@sazzad16 sazzad16 added this to the 4.4.x milestone Oct 18, 2023
@sazzad16 sazzad16 modified the milestones: 4.4.x, 4.4.6 Mar 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for GEOSHAPE index fields and WITHIN and CONTAINS query operators

3 participants