Skip to content

Conversation

andy-stark-redis
Copy link
Collaborator

@andy-stark-redis andy-stark-redis commented Jan 20, 2025

DOC-4733

C# examples for the geo indexing page.

@andy-stark-redis andy-stark-redis self-assigned this Jan 20, 2025
@andy-stark-redis andy-stark-redis added the documentation Improvements or additions to documentation label Jan 20, 2025
Copy link
Collaborator

@atakavci atakavci left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey @andy-stark-redis ,
could u update it as suggested in #391 ?

@andy-stark-redis
Copy link
Collaborator Author

hey @andy-stark-redis , could u update it as suggested in #391 ?

@atakavci Done, but it still seems to fail on the GeoShape examples with the FLAT coordinate system.

@atakavci
Copy link
Collaborator

@andy-stark-redis , i guess it s either

  • the individual server version does not support GEOSHAPE at all,,
  • or its the combination of JSON values with geo coordinates not good on server side

Could you check on those ?

@mgravell
Copy link
Collaborator

mgravell commented Sep 30, 2025

from the CI server:

image

Working fine on server v7 and v8. Does this need a server version-check, like in StreamTutorial ?

@mgravell
Copy link
Collaborator

I'm a little vague on the nuances here, but I think this might tie into RediSearch 2.8 (redis server 7.2 / cluster 7.2.4); source

@andy-stark-redis
Copy link
Collaborator Author

I'm a little vague on the nuances here, but I think this might tie into RediSearch 2.8 (redis server 7.2 / cluster 7.2.4); source

Thanks for the suggestion - just trying it now :-) Amazing if it turns out to be that simple all along!

@mgravell
Copy link
Collaborator

StackExchange.Redis.RedisServerException : geomidx: no such index

Sorry, a bit whackamole

@mgravell
Copy link
Collaborator

(CI goes green) Hoorah!

@andy-stark-redis
Copy link
Collaborator Author

(CI goes green) Hoorah!

Yaay! Thanks very much for your help with this - it's been hanging around for ages, so great to get it fixed.

@andy-stark-redis andy-stark-redis marked this pull request as ready for review September 30, 2025 11:10
@mgravell
Copy link
Collaborator

LGTM; @atakavci does your request for changes still apply?

@atakavci
Copy link
Collaborator

atakavci commented Oct 2, 2025

LGTM; @atakavci does your request for changes still apply?

i see @andy-stark-redis already did the change i mentioned.
It s all good! thanks guys.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants