bgaborg commented on a change in pull request #575: HADOOP-13327 Output Stream
Specification
URL: https://github.com/apache/hadoop/pull/575#discussion_r264164808
##########
File path:
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/fs/Syncable.java
##########
@@ -27,16 +27,16 @@
@InterfaceAudience.Public
@InterfaceStability.Evolving
public interface Syncable {
-
+
/** Flush out the data in client's user buffer. After the return of
* this call, new readers will see the data.
* @throws IOException if any error occurs
*/
- public void hflush() throws IOException;
+ void hflush() throws IOException;
/** Similar to posix fsync, flush out the data in client's user buffer
* all the way to the disk device (but the disk may have it in its cache).
* @throws IOException if error occurs
*/
- public void hsync() throws IOException;
Review comment:
Why do you change the visibility to default/package?
----------------------------------------------------------------
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]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
