KAFKA-3452: follow up - fix state store restoration for session and window stores

[dguy]

The refactoring of the stores in #2166 introduced ChangeLoggingSegmentedBytesStore, thus removing the change logging from the RocksDBWindowStore etc. However, the inner most store still needs to know whether or not logging is enabled such that it can register correctly with ProcessorContext and enable StateStore restoration

[dguy]

@guozhangwang, @mjsax, @enothereska - discovered this today during refactoring of stores. I think this is pretty important!

[asfbot]

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.11/805/
Test PASSed (JDK 8 and Scala 2.11).

[asfbot]

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk8-scala2.12/803/
Test PASSed (JDK 8 and Scala 2.12).

[asfbot]

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/kafka-pr-jdk7-scala2.10/803/
Test PASSed (JDK 7 and Scala 2.10).

[guozhangwang]

@dguy Is this still needed after #2338 is merged? In that case ProcessorStateManager will already know which stores are change-logging enabled so that the second parameter of register(..) is not needed at all.

[dguy]

@guozhangwang - looks like it wont be needed. Thanks for pointing out.