You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/07/14 13:04:15 UTC
[GitHub] [arrow] fredgan opened a new pull request #7754: ARROW-9463:[Go] The writer is double closed in TestReadWrite
fredgan opened a new pull request #7754:
URL: https://github.com/apache/arrow/pull/7754
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [arrow] fredgan commented on pull request #7754: ARROW-9463:[Go] The writer is double closed in TestReadWrite
Posted by GitBox <gi...@apache.org>.
fredgan commented on pull request #7754:
URL: https://github.com/apache/arrow/pull/7754#issuecomment-659776069
@sbinet Hi, would you mind having a look?
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [arrow] fredgan commented on pull request #7754: ARROW-9463:[Go] The writer is double closed in TestReadWrite
Posted by GitBox <gi...@apache.org>.
fredgan commented on pull request #7754:
URL: https://github.com/apache/arrow/pull/7754#issuecomment-659948599
Yes, you are right. It's better than just deleting the redundant Close().
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [arrow] sbinet closed pull request #7754: ARROW-9463: [Go] Make arrjson Writer close idempotent
Posted by GitBox <gi...@apache.org>.
sbinet closed pull request #7754:
URL: https://github.com/apache/arrow/pull/7754
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [arrow] github-actions[bot] commented on pull request #7754: ARROW-9463:[Go] The writer is double closed in TestReadWrite
Posted by GitBox <gi...@apache.org>.
github-actions[bot] commented on pull request #7754:
URL: https://github.com/apache/arrow/pull/7754#issuecomment-658174101
https://issues.apache.org/jira/browse/ARROW-9463
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [arrow] fredgan edited a comment on pull request #7754: ARROW-9463:[Go] The writer is double closed in TestReadWrite
Posted by GitBox <gi...@apache.org>.
fredgan edited a comment on pull request #7754:
URL: https://github.com/apache/arrow/pull/7754#issuecomment-659948599
@sbinet Yes, you are right. It's better than just deleting the redundant Close().
----------------------------------------------------------------
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:
users@infra.apache.org