Add ScanType command to Scan with 'type' option

[forain]

As of version 6.0 you can use this 'type' option to ask SCAN to only
return objects that match a given type, allowing you to
iterate through the database looking for keys of a specific type.

[rbg]

would it be more friendly to perhaps add a TScan command ?

[forain]

would it be more friendly to perhaps add a TScan command ?

Good idea! Is "TScan" the best name for it? We already have SScan for REDIS sscan command and HScan for REDIS hscan command. Having TScan for REDIS "scan command with type option", I'm afraid it might cause confusion.

[rbg]

yeah -- TScan was just a suggestion, @vmihailenco i am sure will have a great suggestion :)

[rbg]

your change as it stands breaks the build so will need some lov; which is why i was suggesting an alternative command..

[forain]

your change as it stands breaks the build so will need some lov; which is why i was suggesting an alternative command..

Fixed the build by updating tests accordingly. Let's wait for @vmihailenco to check if it'd be better to add it to existing Scan as I did or to create a new function with a different name (TScan? TypeScan? ScanWithType?).

[forain]

@rbg @vmihailenco I followed your instructions and reimplemented it as a new "ScanType" command. Also added command and example tests. Build tests passed. Let me know if I missed something!

[vmihailenco]

👍 thanks