[jira] Created: (SANDBOX-310) the multitude of flush in CSVPrinter ain't good

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (SANDBOX-310) the multitude of flush in CSVPrinter ain't good

Gilles (Jira)
the multitude of flush in CSVPrinter ain't good
-----------------------------------------------

                 Key: SANDBOX-310
                 URL: https://issues.apache.org/jira/browse/SANDBOX-310
             Project: Commons Sandbox
          Issue Type: Improvement
            Reporter: Pino Silvaggio


In CSVPrinter too many flushes.

It would be good to add get/setAutoFlush.

Also, I know there is already an open issue discussing this but
it would be really nice to not have 1 single forced flush.
In most cases there is no need of any flush.

Did a benchmark and it does indeed greatly affect performance, which in our cases by 3 times.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (SANDBOX-310) the multitude of flush in CSVPrinter ain't good

Gilles (Jira)

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

Henri Yandell updated SANDBOX-310:
----------------------------------

    Component/s: CSV

> the multitude of flush in CSVPrinter ain't good
> -----------------------------------------------
>
>                 Key: SANDBOX-310
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-310
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: CSV
>            Reporter: Pino Silvaggio
>
> In CSVPrinter too many flushes.
> It would be good to add get/setAutoFlush.
> Also, I know there is already an open issue discussing this but
> it would be really nice to not have 1 single forced flush.
> In most cases there is no need of any flush.
> Did a benchmark and it does indeed greatly affect performance, which in our cases by 3 times.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply | Threaded
Open this post in threaded view
|

[jira] Resolved: (SANDBOX-310) the multitude of flush in CSVPrinter ain't good

Gilles (Jira)
In reply to this post by Gilles (Jira)

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

Yonik Seeley resolved SANDBOX-310.
----------------------------------

    Resolution: Fixed

I noticed the same thing and fixed this as part of SANDBOX-322.

> the multitude of flush in CSVPrinter ain't good
> -----------------------------------------------
>
>                 Key: SANDBOX-310
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-310
>             Project: Commons Sandbox
>          Issue Type: Improvement
>          Components: CSV
>            Reporter: Pino Silvaggio
>
> In CSVPrinter too many flushes.
> It would be good to add get/setAutoFlush.
> Also, I know there is already an open issue discussing this but
> it would be really nice to not have 1 single forced flush.
> In most cases there is no need of any flush.
> Did a benchmark and it does indeed greatly affect performance, which in our cases by 3 times.

--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.