Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-11-18 Thread via GitHub


alamb commented on issue #6591:
URL: https://github.com/apache/arrow-rs/issues/6591#issuecomment-2483441811

   Kudos to @etseidl  -- BTW this should be available on crates.io in about 2 
days -- it is included in 
   - https://github.com/apache/arrow-rs/issues/6597


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-11-16 Thread via GitHub


pacman82 commented on issue #6591:
URL: https://github.com/apache/arrow-rs/issues/6591#issuecomment-2480732129

   Great stuff, 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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-11-16 Thread via GitHub


alamb commented on issue #6591:
URL: https://github.com/apache/arrow-rs/issues/6591#issuecomment-2480597166

   `label_issue.py` automatically added labels {'parquet'} from #6629


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-10-28 Thread via GitHub


tustvold closed issue #6591: Error Instead of Panic On Attempting to Write More 
Than 32769 Row Groups
URL: https://github.com/apache/arrow-rs/issues/6591


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-10-25 Thread via GitHub


alamb commented on issue #6591:
URL: https://github.com/apache/arrow-rs/issues/6591#issuecomment-2438854093

   > Is this fixed by #6378? I can't reproduce with the current head because 
`next_row_group()` will no longer return a rowgroup writer with an invalid 
ordinal.
   
   It appears that #6378 (thanks @progval ❤️ ) was released in version  ` 
53.1.0`
   
https://github.com/apache/arrow-rs/blob/master/CHANGELOG-old.md#5310-2024-10-02
   
   


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-10-25 Thread via GitHub


etseidl commented on issue #6591:
URL: https://github.com/apache/arrow-rs/issues/6591#issuecomment-2438757250

   Is this fixed by #6378? I can't reproduce with the current head because 
`next_row_group()` will no longer return a rowgroup writer with an invalid 
ordinal.


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [I] Error Instead of Panic On Attempting to Write More Than 32769 Row Groups [arrow-rs]

2024-10-20 Thread via GitHub


pacman82 commented on issue #6591:
URL: https://github.com/apache/arrow-rs/issues/6591#issuecomment-2424752485

   No disagreement here. I am exploring opportunities to change the UX of 
`odbc2parquet` in a way to avoid this scenario entirely, but still felt that 
the panic should be an error.


-- 
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-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org