You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@arrow.apache.org by "KarateSnowMachine (via GitHub)" <gi...@apache.org> on 2023/06/20 20:52:18 UTC
[GitHub] [arrow] KarateSnowMachine opened a new issue, #36189: [C++] Parquet StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
KarateSnowMachine opened a new issue, #36189:
URL: https://github.com/apache/arrow/issues/36189
### Describe the bug, including details regarding any error messages, version, and platform.
The issue arises due to a simple arithmetic error when calculating the offset from the current row group. As far as I can tell this affects all platforms and the bug has existed since the introduction of the StreamReader skip code in https://github.com/apache/arrow/commit/56adb6a7ecd5f189f03f4a73a6df1c9a2af1eeb0
I will submit a PR with a unit test case and proposed fix shortly.
### Component(s)
C++
--
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-unsubscribe@arrow.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] wgtmac closed issue #36189: [C++][Parquet] StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
Posted by "wgtmac (via GitHub)" <gi...@apache.org>.
wgtmac closed issue #36189: [C++][Parquet] StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
URL: https://github.com/apache/arrow/issues/36189
--
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-unsubscribe@arrow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [arrow] KarateSnowMachine commented on issue #36189: [C++] Parquet StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
Posted by "KarateSnowMachine (via GitHub)" <gi...@apache.org>.
KarateSnowMachine commented on issue #36189:
URL: https://github.com/apache/arrow/issues/36189#issuecomment-1599520187
OK, maybe I didn't read the contributing guidelines correctly but I wasn't successful in getting my PR to automatically reference this issue. It is here: https://github.com/apache/arrow/pull/36191
--
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] wgtmac commented on issue #36189: [C++][Parquet] StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
Posted by "wgtmac (via GitHub)" <gi...@apache.org>.
wgtmac commented on issue #36189:
URL: https://github.com/apache/arrow/issues/36189#issuecomment-1601075250
> > > OK, maybe I didn't read the contributing guidelines correctly but I wasn't successful in getting my PR to automatically reference this issue. It is here: #36191
> >
> >
> > The PR title has to be started with `GH-XXX` to link it to correct issue.
>
> Should I submit a patch for the contributing guide, which says the issue should be named ARROW-XXX or PARQUET-XXXX? https://github.com/apache/arrow/blob/main/.github/CONTRIBUTING.md?plain=1#L61
Yes, that would be great!
--
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] wgtmac commented on issue #36189: [C++] Parquet StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
Posted by "wgtmac (via GitHub)" <gi...@apache.org>.
wgtmac commented on issue #36189:
URL: https://github.com/apache/arrow/issues/36189#issuecomment-1600090683
> OK, maybe I didn't read the contributing guidelines correctly but I wasn't successful in getting my PR to automatically reference this issue. It is here: #36191
The PR title has to be started with `GH-XXX` to link it to correct issue.
--
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] KarateSnowMachine commented on issue #36189: [C++][Parquet] StreamReader::SkipRows() skips to wrong spot in files containing multiple row groups
Posted by "KarateSnowMachine (via GitHub)" <gi...@apache.org>.
KarateSnowMachine commented on issue #36189:
URL: https://github.com/apache/arrow/issues/36189#issuecomment-1601005723
> > OK, maybe I didn't read the contributing guidelines correctly but I wasn't successful in getting my PR to automatically reference this issue. It is here: #36191
>
> The PR title has to be started with `GH-XXX` to link it to correct issue.
Should I submit a patch for the contributing guide, which says the issue should be named ARROW-XXX or PARQUET-XXXX?
https://github.com/apache/arrow/blob/main/.github/CONTRIBUTING.md?plain=1#L61
--
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