[GitHub] [commons-csv] garydgregory commented on pull request #162: CSV-278: Reuse Buffers in Lexer for Delimiter Detection

2021-07-14 Thread GitBox


garydgregory commented on pull request #162:
URL: https://github.com/apache/commons-csv/pull/162#issuecomment-880044633


   Maybe tonight or this weekend.


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-csv] garydgregory commented on pull request #162: CSV-278: Reuse Buffers in Lexer for Delimiter Detection

2021-07-13 Thread GitBox


garydgregory commented on pull request #162:
URL: https://github.com/apache/commons-csv/pull/162#issuecomment-879529981


   @belugabehr 
   Please provide JMH numbers for this PR if you are providing this for 
performance reasons. Otherwise, it's just  a change to complex code not backed 
up by anything. 
   If you want to put more in this PR that is fine as well.
   TY


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-csv] garydgregory commented on pull request #162: CSV-278: Reuse Buffers in Lexer for Delimiter Detection

2021-07-13 Thread GitBox


garydgregory commented on pull request #162:
URL: https://github.com/apache/commons-csv/pull/162#issuecomment-879491310


   @belugabehr 
   May you please resolve conflicts and post your JMH benchmark results?
   TY!


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-csv] garydgregory commented on pull request #162: CSV-278: Reuse Buffers in Lexer for Delimiter Detection

2021-07-13 Thread GitBox


garydgregory commented on pull request #162:
URL: https://github.com/apache/commons-csv/pull/162#issuecomment-879161545


   This PR has no description or URL to the Jira ticket.


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [commons-csv] garydgregory commented on pull request #162: CSV-278: Reuse Buffers in Lexer for Delimiter Detection

2021-07-13 Thread GitBox


garydgregory commented on pull request #162:
URL: https://github.com/apache/commons-csv/pull/162#issuecomment-879161167


   This PR has no description or URL to the Jira ticket.


-- 
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: issues-unsubscr...@commons.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org