[Go][Parquet] ColumnWriter TotalCompressedBytes will be corrupt if FallbackToPlain #39921
Comments
mapleFU
added a commit
to mapleFU/arrow
that referenced
this issue
Feb 2, 2024
zeroshade
pushed a commit
that referenced
this issue
Feb 5, 2024
…fter Flush (#39922) ### Rationale for this change See #39921 ### What changes are included in this PR? Not clearing `totalCompressedBytes` when flush called ### Are these changes tested? Yes ### Are there any user-facing changes? Yes, it's a bugfix * Closes: #39921 Authored-by: mwish <[email protected]> Signed-off-by: Matt Topol <[email protected]>
dgreiss
pushed a commit
to dgreiss/arrow
that referenced
this issue
Feb 19, 2024
…ytes after Flush (apache#39922) ### Rationale for this change See apache#39921 ### What changes are included in this PR? Not clearing `totalCompressedBytes` when flush called ### Are these changes tested? Yes ### Are there any user-facing changes? Yes, it's a bugfix * Closes: apache#39921 Authored-by: mwish <[email protected]> Signed-off-by: Matt Topol <[email protected]>
zanmato1984
pushed a commit
to zanmato1984/arrow
that referenced
this issue
Feb 28, 2024
…ytes after Flush (apache#39922) ### Rationale for this change See apache#39921 ### What changes are included in this PR? Not clearing `totalCompressedBytes` when flush called ### Are these changes tested? Yes ### Are there any user-facing changes? Yes, it's a bugfix * Closes: apache#39921 Authored-by: mwish <[email protected]> Signed-off-by: Matt Topol <[email protected]>
thisisnic
pushed a commit
to thisisnic/arrow
that referenced
this issue
Mar 8, 2024
…ytes after Flush (apache#39922) ### Rationale for this change See apache#39921 ### What changes are included in this PR? Not clearing `totalCompressedBytes` when flush called ### Are these changes tested? Yes ### Are there any user-facing changes? Yes, it's a bugfix * Closes: apache#39921 Authored-by: mwish <[email protected]> Signed-off-by: Matt Topol <[email protected]>
kou
pushed a commit
to apache/arrow-go
that referenced
this issue
Aug 30, 2024
…fter Flush (#39922) ### Rationale for this change See apache/arrow#39921 ### What changes are included in this PR? Not clearing `totalCompressedBytes` when flush called ### Are these changes tested? Yes ### Are there any user-facing changes? Yes, it's a bugfix * Closes: #39921 Authored-by: mwish <[email protected]> Signed-off-by: Matt Topol <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug, including details regarding any error messages, version, and platform.
See #39870
Component(s)
Go, Parquet
The text was updated successfully, but these errors were encountered: