[jira] [Updated] (HADOOP-15962) FileUtils Small Buffer Size

2018-12-27 Thread BELUGA BEHR (JIRA)


 [ 
https://issues.apache.org/jira/browse/HADOOP-15962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

BELUGA BEHR updated HADOOP-15962:
-
Attachment: HADOOP-15962.2.patch

> FileUtils Small Buffer Size
> ---
>
> Key: HADOOP-15962
> URL: https://issues.apache.org/jira/browse/HADOOP-15962
> Project: Hadoop Common
>  Issue Type: Improvement
>  Components: common
>Affects Versions: 3.3.0
>Reporter: BELUGA BEHR
>Assignee: BELUGA BEHR
>Priority: Minor
> Attachments: HADOOP-15962.1.patch, HADOOP-15962.2.patch
>
>
> Note sure if this code is even being used, but it implements a copy routing 
> utilizing a 2K buffer.  Modern JVM uses 8K, but 4K should be minimum.  Also, 
> there are libraries for this stuff.
> {code:java|title=FileUtil.java}
> int count;
> byte data[] = new byte[2048];
> try (BufferedOutputStream outputStream = new BufferedOutputStream(
> new FileOutputStream(outputFile));) {
>   while ((count = tis.read(data)) != -1) {
> outputStream.write(data, 0, count);
>   }
>   outputStream.flush();
> }
> {code}
> I also fixed a couple of check-style warnings.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



[jira] [Updated] (HADOOP-15962) FileUtils Small Buffer Size

2018-12-27 Thread BELUGA BEHR (JIRA)


 [ 
https://issues.apache.org/jira/browse/HADOOP-15962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

BELUGA BEHR updated HADOOP-15962:
-
Status: Open  (was: Patch Available)

> FileUtils Small Buffer Size
> ---
>
> Key: HADOOP-15962
> URL: https://issues.apache.org/jira/browse/HADOOP-15962
> Project: Hadoop Common
>  Issue Type: Improvement
>  Components: common
>Affects Versions: 3.3.0
>Reporter: BELUGA BEHR
>Assignee: BELUGA BEHR
>Priority: Minor
> Attachments: HADOOP-15962.1.patch, HADOOP-15962.2.patch
>
>
> Note sure if this code is even being used, but it implements a copy routing 
> utilizing a 2K buffer.  Modern JVM uses 8K, but 4K should be minimum.  Also, 
> there are libraries for this stuff.
> {code:java|title=FileUtil.java}
> int count;
> byte data[] = new byte[2048];
> try (BufferedOutputStream outputStream = new BufferedOutputStream(
> new FileOutputStream(outputFile));) {
>   while ((count = tis.read(data)) != -1) {
> outputStream.write(data, 0, count);
>   }
>   outputStream.flush();
> }
> {code}
> I also fixed a couple of check-style warnings.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



[jira] [Updated] (HADOOP-15962) FileUtils Small Buffer Size

2018-12-27 Thread BELUGA BEHR (JIRA)


 [ 
https://issues.apache.org/jira/browse/HADOOP-15962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

BELUGA BEHR updated HADOOP-15962:
-
Status: Patch Available  (was: Open)

[~ste...@apache.org] Thanks for the review!

I have removed some changes for easier merging.

> FileUtils Small Buffer Size
> ---
>
> Key: HADOOP-15962
> URL: https://issues.apache.org/jira/browse/HADOOP-15962
> Project: Hadoop Common
>  Issue Type: Improvement
>  Components: common
>Affects Versions: 3.3.0
>Reporter: BELUGA BEHR
>Assignee: BELUGA BEHR
>Priority: Minor
> Attachments: HADOOP-15962.1.patch, HADOOP-15962.2.patch
>
>
> Note sure if this code is even being used, but it implements a copy routing 
> utilizing a 2K buffer.  Modern JVM uses 8K, but 4K should be minimum.  Also, 
> there are libraries for this stuff.
> {code:java|title=FileUtil.java}
> int count;
> byte data[] = new byte[2048];
> try (BufferedOutputStream outputStream = new BufferedOutputStream(
> new FileOutputStream(outputFile));) {
>   while ((count = tis.read(data)) != -1) {
> outputStream.write(data, 0, count);
>   }
>   outputStream.flush();
> }
> {code}
> I also fixed a couple of check-style warnings.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



[jira] [Updated] (HADOOP-15962) FileUtils Small Buffer Size

2018-11-30 Thread BELUGA BEHR (JIRA)


 [ 
https://issues.apache.org/jira/browse/HADOOP-15962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

BELUGA BEHR updated HADOOP-15962:
-
Status: Patch Available  (was: Open)

> FileUtils Small Buffer Size
> ---
>
> Key: HADOOP-15962
> URL: https://issues.apache.org/jira/browse/HADOOP-15962
> Project: Hadoop Common
>  Issue Type: Improvement
>  Components: common
>Affects Versions: 3.3.0
>Reporter: BELUGA BEHR
>Assignee: BELUGA BEHR
>Priority: Minor
> Attachments: HADOOP-15962.1.patch
>
>
> Note sure if this code is even being used, but it implements a copy routing 
> utilizing a 2K buffer.  Modern JVM uses 8K, but 4K should be minimum.  Also, 
> there are libraries for this stuff.
> {code:java|title=FileUtil.java}
> int count;
> byte data[] = new byte[2048];
> try (BufferedOutputStream outputStream = new BufferedOutputStream(
> new FileOutputStream(outputFile));) {
>   while ((count = tis.read(data)) != -1) {
> outputStream.write(data, 0, count);
>   }
>   outputStream.flush();
> }
> {code}
> I also fixed a couple of check-style warnings.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org



[jira] [Updated] (HADOOP-15962) FileUtils Small Buffer Size

2018-11-30 Thread BELUGA BEHR (JIRA)


 [ 
https://issues.apache.org/jira/browse/HADOOP-15962?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

BELUGA BEHR updated HADOOP-15962:
-
Attachment: HADOOP-15962.1.patch

> FileUtils Small Buffer Size
> ---
>
> Key: HADOOP-15962
> URL: https://issues.apache.org/jira/browse/HADOOP-15962
> Project: Hadoop Common
>  Issue Type: Improvement
>  Components: common
>Affects Versions: 3.3.0
>Reporter: BELUGA BEHR
>Assignee: BELUGA BEHR
>Priority: Minor
> Attachments: HADOOP-15962.1.patch
>
>
> Note sure if this code is even being used, but it implements a copy routing 
> utilizing a 2K buffer.  Modern JVM uses 8K, but 4K should be minimum.  Also, 
> there are libraries for this stuff.
> {code:java|title=FileUtil.java}
> int count;
> byte data[] = new byte[2048];
> try (BufferedOutputStream outputStream = new BufferedOutputStream(
> new FileOutputStream(outputFile));) {
>   while ((count = tis.read(data)) != -1) {
> outputStream.write(data, 0, count);
>   }
>   outputStream.flush();
> }
> {code}
> I also fixed a couple of check-style warnings.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

-
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org