ShivamSharma43 commented on issue #2082:
URL: https://github.com/apache/iceberg-go/issues/2082#issuecomment-6577063551

   Hi @eneskeles, are you still planning to work on this? If you've moved on to 
other things, I'd be happy to pick it up. No pressure if you're still on it, 
though.
   
   My plan would be to follow the issue description:
   
   - Replace the `logFatal("unimplemented output type")` default arm in 
`cmd/iceberg/main.go` with an error that names the unsupported value and lists 
the supported types (`text`, `json`).
   - Exit non-zero the same way the rollback-selector and DynamoDB checks just 
below it do.
   - Add a test using the existing `osExit` hook in `cmd/iceberg`.
   
   I'll wait to hear from you or a maintainer before starting.
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to