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

Add support for RedisJSON #657

Merged
merged 29 commits into from Aug 25, 2022
Merged

Add support for RedisJSON #657

merged 29 commits into from Aug 25, 2022

Commits on Jul 29, 2022

  1. moved the implement macro to allow for multiple calls of it in separa… 

    …te files
    @d3rpp
    d3rpp committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    e777183 View commit details
    Browse the repository at this point in the history

  2. removed old macro definition

    @d3rpp
    d3rpp committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    5403c6c View commit details
    Browse the repository at this point in the history

  3. added extra newline

    @d3rpp
    d3rpp committed Jul 29, 2022
    Configuration menu
    Copy the full SHA
    e835779 View commit details
    Browse the repository at this point in the history

Commits on Jul 30, 2022

  1. added redisjson support 

    added extra newline

Replaced json_mget with condition in json_get

removed call to std::mem::replace in the macro, it wasn't doing anything and all tests still passs

added newlines

Add a Discord badge to README

hid internal function from docs

formatted
    @d3rpp
    d3rpp committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    6cf2dd5 View commit details
    Browse the repository at this point in the history

  2. Ran cargo fmt

    @d3rpp
    d3rpp committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    6130290 View commit details
    Browse the repository at this point in the history

  3. added redisjson support 

    added extra newline

Replaced json_mget with condition in json_get

removed call to std::mem::replace in the macro, it wasn't doing anything and all tests still passs

added newlines

Add a Discord badge to README

hid internal function from docs

formatted

Ran cargo fmt

Add a Discord badge to README
    @d3rpp
    d3rpp committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    3ee15e0 View commit details
    Browse the repository at this point in the history

  4. Merge pull request #7 from redis-rs/main 

    Add a Discord badge to README
    @d3rpp
    d3rpp committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    f8a849f View commit details
    Browse the repository at this point in the history

  5. Merge pull request #8 from redis-rs/main 

    Add a Discord badge to README
    @d3rpp
    d3rpp committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    c83ce0a View commit details
    Browse the repository at this point in the history

  6. Merge pull request #9 from d3rpp/json_support 

    Json support to local main
    @d3rpp
    d3rpp committed Jul 30, 2022
    Configuration menu
    Copy the full SHA
    1e392f8 View commit details
    Browse the repository at this point in the history

Commits on Jul 31, 2022

  1. removed unecessary call to std::mem::replace

    @d3rpp
    d3rpp committed Jul 31, 2022
    Configuration menu
    Copy the full SHA
    dc2543a View commit details
    Browse the repository at this point in the history

Commits on Aug 2, 2022

  1. Merge branch 'main' of github.com:redis-rs/redis-rs into redis-rs-main

    @d3rpp
    d3rpp committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    c5b29f8 View commit details
    Browse the repository at this point in the history

  2. Merge branch 'redis-rs-main' 

    merging from redis-rs/redis-rs#main into d3rpp/redis-rs-tokio-drift#main
    @d3rpp
    d3rpp committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    7b7185c View commit details
    Browse the repository at this point in the history

  3. Code Review changes 

    - Updated README to use correct module import
- Normalized spelling of Serialization
- implemented `From<serde_json::Error> for RedisError` to make implementation of some commands, cleaner.
- updated command names to be more readable (json_strappend -> json_str_append, etc.)
    @d3rpp
    d3rpp committed Aug 2, 2022
    Configuration menu
    Copy the full SHA
    18ffcd4 View commit details
    Browse the repository at this point in the history

Commits on Aug 4, 2022

  1. Update README.md 

    Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    e52335f View commit details
    Browse the repository at this point in the history

  2. Update redis/src/commands/json.rs 

    removing extra "s" due to me misreading the redis docs

Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    3d983f9 View commit details
    Browse the repository at this point in the history

  3. Update redis/src/commands/json.rs 

    updated comment

TODO: RE-ADD START + STOP

Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    c372c3e View commit details
    Browse the repository at this point in the history

  4. Update redis/src/commands/json.rs 

    updated comments for `json_arr_insert`

Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    481cc0c View commit details
    Browse the repository at this point in the history

  5. Updated comment for json_arr_pop 

    Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    0851ef2 View commit details
    Browse the repository at this point in the history

  6. Update docs for json_arr_get 

    Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    a700fe8 View commit details
    Browse the repository at this point in the history

  7. Accounted for style nit in json_set 

    Co-authored-by: Dirkjan Ochtman <dirkjan@ochtman.nl>
    @d3rpp @djc
    d3rpp and djc committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    521b91c View commit details
    Browse the repository at this point in the history

  8. Updated readme to use ToRedisArgs instead of ToString

    @d3rpp
    d3rpp committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    220bad3 View commit details
    Browse the repository at this point in the history

  9. > Nit: let's keep the spawner last, I think that looks a little cle… 

    …aner.

- djc
    @d3rpp
    d3rpp committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    8817a15 View commit details
    Browse the repository at this point in the history

  10. Updated Cmd::arg_take to have a more specific and explanatory comment

    @d3rpp
    d3rpp committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    037e8ff View commit details
    Browse the repository at this point in the history

  11. ran cargo fmt --all

    @d3rpp
    d3rpp committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    6ebd9d3 View commit details
    Browse the repository at this point in the history

  12. Added start and stop to json_arr_insert by making `json_arr_insert_… 

    …ss` to make it cleaner
    @d3rpp
    d3rpp committed Aug 4, 2022
    Configuration menu
    Copy the full SHA
    6f05664 View commit details
    Browse the repository at this point in the history

Commits on Aug 16, 2022

  1. Merge branch 'redis-rs:main' into main

    @d3rpp
    d3rpp committed Aug 16, 2022
    Configuration menu
    Copy the full SHA
    c7ef25a View commit details
    Browse the repository at this point in the history

Commits on Aug 23, 2022

  1. reformatted json commands 

    removed `arg_take`
    @d3rpp
    d3rpp committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    bebbba8 View commit details
    Browse the repository at this point in the history

  2. Moved SerdeJSON Error to RedisError definition to types.rs 

    Moved json macro declaration to json.rs

Renamed RedisJson and RedisModule to Json and Module respectfully
    @d3rpp
    d3rpp committed Aug 23, 2022
    Configuration menu
    Copy the full SHA
    4a13398 View commit details
    Browse the repository at this point in the history

Commits on Aug 25, 2022

  1. Merge branch 'redis-rs:main' into main

    @d3rpp
    d3rpp committed Aug 25, 2022
    Configuration menu
    Copy the full SHA
    cf5a937 View commit details
    Browse the repository at this point in the history