Adds cache name to Redis store cache statements to ease debugging #7

vdzhuvinov · 2016-11-10T09:17:01Z

No description provided.

tristantarrant · 2016-12-05T08:27:23Z

src/main/java/org/infinispan/persistence/redis/RedisStore.java

@@ -37,7 +37,7 @@
    @Override
    public void init(InitializationContext ctx)
    {
-        RedisStore.log.info("Redis cache store initialising");
+        RedisStore.log.info("Initialising Redis store for cache " + ctx.getCache().getName());


Better to use:
RedisStore.log.info("Initialising Redis store for cache '%s' ", ctx.getCache().getName());

Cheers, I'll get to update the PR later this week.

vdzhuvinov added 2 commits November 10, 2016 10:41

Fixes parsing of top-level inherited XML attributes (iss infinispan#4)

9a0094b

Includes cache name in Redis store log statements (iss infinispan#6)

52f6de0

tristantarrant reviewed Dec 5, 2016

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Adds cache name to Redis store cache statements to ease debugging #7

Adds cache name to Redis store cache statements to ease debugging #7

vdzhuvinov commented Nov 10, 2016

tristantarrant Dec 5, 2016

vdzhuvinov Dec 5, 2016

Adds cache name to Redis store cache statements to ease debugging #7

Are you sure you want to change the base?

Adds cache name to Redis store cache statements to ease debugging #7

Conversation

vdzhuvinov commented Nov 10, 2016

tristantarrant Dec 5, 2016

Choose a reason for hiding this comment

vdzhuvinov Dec 5, 2016

Choose a reason for hiding this comment