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.