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 2022/11/01 21:07:59 UTC
[GitHub] [arrow-julia] baumgold opened a new pull request, #356: fix BatchIterator iterate method to handle partial messages
baumgold opened a new pull request, #356:
URL: https://github.com/apache/arrow-julia/pull/356
The current implementation blindly assumes the message is complete and can cause undefined behavior if it is not. This fixes the iterate method to ignore partial messages.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-julia] quinnj merged pull request #356: fix BatchIterator iterate method to handle partial messages
Posted by GitBox <gi...@apache.org>.
quinnj merged PR #356:
URL: https://github.com/apache/arrow-julia/pull/356
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-julia] kou commented on pull request #356: fix BatchIterator iterate method to handle partial messages
Posted by GitBox <gi...@apache.org>.
kou commented on PR #356:
URL: https://github.com/apache/arrow-julia/pull/356#issuecomment-1301322901
@baumgold Could you see my e-mail reply for your Apache account? You need to configure https://gitbox.apache.org/ .
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-julia] baumgold commented on pull request #356: fix BatchIterator iterate method to handle partial messages
Posted by GitBox <gi...@apache.org>.
baumgold commented on PR #356:
URL: https://github.com/apache/arrow-julia/pull/356#issuecomment-1301353967
> @baumgold Could you see my e-mail reply for your Apache account? You need to configure https://gitbox.apache.org/ .
Got it. Configured gitbox. All is well with the world. Many thanks.
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow-julia] baumgold commented on pull request #356: fix BatchIterator iterate method to handle partial messages
Posted by GitBox <gi...@apache.org>.
baumgold commented on PR #356:
URL: https://github.com/apache/arrow-julia/pull/356#issuecomment-1301313046
@quinnj - thanks for reviewing/approving. I still don't have write access (working on getting permissions from Apache). In the mean time would you mind merging for me, please? I'd like to propose a new release (again).
--
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: github-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org