timoninmaxim commented on a change in pull request #9215:
URL: https://github.com/apache/ignite/pull/9215#discussion_r663747590
##########
File path:
modules/indexing/src/test/java/org/apache/ignite/internal/processors/query/SqlSystemViewsSelfTest.java
##########
@@ -1567,7 +1567,6 @@ public void testCachesViews() throws Exception {
"ON C.CACHE_NAME = CG.CACHE_GROUP_NAME WHERE C.CACHE_NAME like
'cache%' " +
"AND C.CACHE_MODE = CG.CACHE_MODE " +
"AND C.ATOMICITY_MODE = CG.ATOMICITY_MODE " +
- "AND COALESCE(C.AFFINITY, '-') = COALESCE(CG.AFFINITY, '-') " +
Review comment:
Sorry, I don't found lastWakeUpTime in affinity functions, could you
please provide a link? I see that AffinityFunction has static fields only, like
`partitions`, `backups`.
The problem I saw that RendevouzAffinityFunction has Ignite instance in
toString. And we return it to user when it ask for affintiy function from view.
Looks incorrect, isn't it?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
