Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

LZO and LZO_RAW Support #18

Open
shannonwells opened this issue Jun 29, 2021 · 0 comments
Open

LZO and LZO_RAW Support #18

shannonwells opened this issue Jun 29, 2021 · 0 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@shannonwells
Copy link
Collaborator

Steps to reproduce

Run the LZO tests in test/integration.js

Expected behaviour

The tests should pass

Actual behaviour

The round-trip test fails with:

Error: Decompression failed with code: LZO_E_OUTPUT_OVERRUN
    at Object.decompress (node_modules/lzo/index.js:59:13)
    at Object.inflate_lzo [as inflate] (lib/compression.js:91:14)
    at Object.inflate (lib/compression.js:75:52)
    at decodeDataPageV2 (lib/reader.js:932:47)
    at decodePage (lib/reader.js:710:20)
    at decodePages (lib/reader.js:747:28)
    at /Users/shannonwells/github.com/ProjectLiberty/parquetjs/lib/reader.js:609:85
    at ParquetEnvelopeReader.readRowGroup (lib/reader.js:567:35)
    at ParquetCursor.next (lib/reader.js:67:23)
    at readTestFile (test/integration.js:293:24)
@wilwade wilwade added this to the Q2/Q3 2024 Improvements milestone Jun 11, 2024
@wilwade wilwade self-assigned this Jul 2, 2024
@wilwade wilwade removed this from the Q2/Q3 2024 Improvements milestone Sep 12, 2024
@wilwade wilwade added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed labels Sep 12, 2024
@wilwade wilwade removed their assignment Sep 12, 2024
@wilwade wilwade changed the title LZO write/read test fails withDecompression failed with code: LZO_E_OUTPUT_OVERRUN LZO and LZO_RAW Support Sep 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants