Comment From: ZhangShenao
I extracted a method for parsing cache value in BaseExecutor, and it can be reused by other classes, please review.
Comment From: ZhangShenao
I extracted a method for parsing cache value in BaseExecutor, and it can be reused by other classes, please review.
Comment From: harawata
Thank you for the PR, @ZhangShenao !
These lines look the same, but I am not sure if it's right to extract and share that particular part. And by making it a public static method, we have to worry about backward compatibility in future development. We might reconsider it in the future, but I would like to keep it as-is for now.