Map storage looks up ID via the session, that is no longer needed

[ahus1]

Describe the bug

The following example first looks up the realmId, then looks it up via the session again.

keycloak/model/map/src/main/java/org/keycloak/models/map/role/MapRoleProvider.java

Lines 194 to 200 in fc974fc

	String roleId = tx.read(withCriteria(mcb))
	.map(entityToAdapterFunc(realm))
	.map(RoleModel::getId)
	.findFirst()
	.orElse(null);
	//we need to go via session.roles() not to bypass cache
	return roleId == null ? null : session.roles().getRoleById(realm, roleId);

Such a behavior will lead to problem in a tree store, where a partial store should return only its partial results.

Version

999-SNAPSHOT

Expected behavior

As no caches are in place outside of the map store, this behavior is now no longer needed. All legacy caches are disabled.

Actual behavior

Additional lookup by ID