We shouldn't put policy level configurations here because it makes it less pluggable (and this should be a table level setting really). Can we make this change?

In table property, the CachePolicy should also specify the parameters, .e.g

"shark.cache.partition.cachePolicy" = "shark.memstore2.LRACachePolicy(30)"

When we get the cache policy string, let's find the policy part (before parenthesis) and instantiate the policy, and leave the parts in parenthesis to the cache policy itself to parse.