You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2021/09/16 16:09:50 UTC

[GitHub] [geode] DonalEvans commented on pull request #6870: GEODE-9582: make radish pattern match compatible with redis

DonalEvans commented on pull request #6870:
URL: https://github.com/apache/geode/pull/6870#issuecomment-921037920


   > @DonalEvans The native redis code that parses the glob pattern does nothing with quotes. So I think what you are seeing is something redis-cli adds. If a double or single quote end up getting passed in as part of the pattern it will just be treated as a literal character. The ones I added to the unit test were to make sure all the parsing code had coverage. I can add those also to the existing glob integration test
   
   Ah, thanks for the explanation. Given that, I think the existing coverage is sufficient.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@geode.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org