zeroshade commented on issue #31:
URL: https://github.com/apache/arrow-go/issues/31#issuecomment-5501458581

   Removing the `good-first-issue` label here — this issue stays open and is 
still valid, it's just not a starter task.
   
   The misleading doc comment on `WithEncoding` is a one-line fix, but the 
actual feature is not: the fallback is hardcoded in `FallbackToPlain`, which 
lives in `parquet/file/column_writer_types.gen.go` and is generated from 
`column_writer_types.gen.go.tmpl`. Doing this properly means editing the 
template and regenerating, plus deciding which encodings are legal fallbacks 
for each physical type and rejecting the rest. That's a reasonable intermediate 
task but a rough first contribution.
   
   Still very much open to a PR from anyone who wants it.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to