ivanzlenko commented on code in PR #7374:
URL: https://github.com/apache/ignite-3/pull/7374#discussion_r2676273003
##########
modules/page-memory/src/main/java/org/apache/ignite/internal/pagememory/DataRegion.java:
##########
@@ -25,4 +25,9 @@ public interface DataRegion<T extends PageMemory> {
* Returns page memory.
*/
T pageMemory();
+
+ /**
+ * Returns the region size in bytes.
+ */
+ long regionSize();
Review Comment:
Because it goes against everything interfaces stood for.
If we do not use this method in tests - it is okay. Modify production code
to cater to test implementation is a very bad practice.
--
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]
