ibessonov commented on a change in pull request #7941:
URL: https://github.com/apache/ignite/pull/7941#discussion_r502276110
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/tree/io/PageIO.java
##########
@@ -258,6 +258,9 @@
/** */
public static final short T_MARKER_PAGE = 33;
+ /** */
+ public static final short T_INDEX_META = 34;
Review comment:
Yes, this is correct. Naming of the classes doesn't bother me too much
so I was fine with "Indexing" instead of V2 (although that page is also
referred as "cache group meta page" sometimes, like in PageMemoryEx#metaPageId,
for example).
I think I understand the core problem, inheritance between MetaIO and
PartitionMetaIO should have been replaced with extension of the same abstract
superclass back in the day. But with current faulty design people just get
confused :(
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
