PragmaTwice opened a new pull request, #1032: URL: https://github.com/apache/incubator-kvrocks/pull/1032
The current `CommandParser` is a prototype that adds only a few methods that are really usable at the moment (but does not affect it being merged). To demonstrate the use of `CommandParser`, I rewrote the `Parse` implementation for a few commands in `redis_cmd.cc`. It is easy to see that the amount of code is massively (multiplied) reduced, and the parsing logic can be expressed in just a few lines of code, especially when complex-syntax commands are encountered. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
