Vladsz83 commented on a change in pull request #9269:
URL: https://github.com/apache/ignite/pull/9269#discussion_r689478212
##########
File path:
modules/core/src/main/java/org/apache/ignite/internal/processors/cache/persistence/file/EncryptedFileIO.java
##########
@@ -67,15 +67,15 @@
* @param plainFileIO Underlying file.
* @param groupId Group id.
* @param pageSize Size of plain data page in bytes.
- * @param headerSize Size of file header in bytes.
+ * @param plainHeader Size of file header in bytes which is never
encrypted.
Review comment:
>>> "If the read() method is forbidden, so what is a reason to change
it? Could we throw an exception there instead?"
The reason is the usage of the method. It is invoked sometimes with zero
offset.
--
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]