Reduce allocations from Redis operations

[theigl]

RedisIndexedSessionRepository and RedisSessionExpirationPolicy create a lot of single-use proxies by relying on BoundHashOperations.

For instance in this case:

spring-session/spring-session-data-redis/src/main/java/org/springframework/session/data/redis/RedisIndexedSessionRepository.java

Lines 898 to 903 in f6f4c86

	private void saveDelta() {
	if (this.delta.isEmpty()) {
	return;
	}
	String sessionId = getId();
	getSessionBoundHashOperations(sessionId).putAll(this.delta);

spring-session/spring-session-data-redis/src/main/java/org/springframework/session/data/redis/RedisIndexedSessionRepository.java

Lines 733 to 735 in f6f4c86

	private BoundHashOperations<String, String, Object> getSessionBoundHashOperations(String sessionId) {
	String key = getSessionKey(sessionId);
	return this.sessionRedisOperations.boundHashOps(key);

This showed up in our production profiler because of hundreds of MB allocated in java.lang.reflect.Method:

If I understand the code correctly, this is done purely for convenience and could simply be replaced by the following code:

String key = getSessionKey(sessionId);
sessionRedisOperations.opsForHash().putAll(key, this.delta);

Or am I overlooking something?