Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
    Mark Nunberg
    @mnunberg
    welcome all
    Guy Korland
    @gkorland
    Welcome @mnunberg
    Guy Korland
    @gkorland
    Pieter Cailliau
    @K-Jo
    We released our second RC for RediSearch 1.6, release notes can be found here: https://github.com/RediSearch/RediSearch/releases/tag/v1.6.3 All feedback is welcome!
    Bruno Bigras
    @bbigras
    Is it possible to filter using another query? I might not be using the correct terms. Let say I have a list of cars and a list of cars brands. I would like to filter the cars by model but also on some property of the cars brands.
    Guy Korland
    @gkorland
    There are two options
    1. De-normalize the schema and include the brand data in each car
    2. Use another module that utilize RediSearch but adds support for relations like RedisGraph
    Adam
    @cheeseandcereal

    Hey, is there any possibility of some PRs getting merged into redisearch-py ? Particularly these 2: RediSearch/redisearch-py#38 RediSearch/redisearch-py#37
    Also I just ran into this issue myself: RediSearch/redisearch-py#31 and I would be willing to put in the work to do a PR to fix this, but only if I knew it was going to be accepted. Considering I put in a PR back in August that still hasn't been merged, I don't have great faith that if I put in the work, anything will actually get merged and released at the moment.

    Without that commitment, I'll just have to fork and maintain my own version of redisearch-py, which I'd really rather not have to do if I don't need to. I would love for these fixes/features to be available to everyone

    Adam
    @cheeseandcereal
    Turns out fixing that bug was really simple: RediSearch/redisearch-py#44
    It would be great if those 3 PRs could be merged/released....
    Meir Shpilraien (Spielrein)
    @MeirShpilraien
    @cheeseandcereal tha
    @cheeseandcereal thank you for you contribution and I am sorry it was not merged yet (we probably missed it so it is good you wrote this message). I will personally make sure that me (or someone else) will review your PR's and we will defenatly merge them. Any contribution is always welcome.
    Adam
    @cheeseandcereal
    I really appreciate you guys merging those PRs! Is there any chance that a release could be made for what's on master at the moment? It has a decent amount of changes since the least release ~1.5 years ago including those useful bug fixes.
    Guy Korland
    @gkorland
    we'll probbaly release a new version soon
    Adam
    @cheeseandcereal
    That would be awesome! Thanks!
    Guy Korland
    @gkorland
    Adam
    @cheeseandcereal
    🎉