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.

Sign up for GitHub

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

Jump to bottom

[SIG-40820]don't panic if type mismatch is decode arrow #154

Merged

Yifeng-Sigma merged 2 commits into master from yifeng/no_panic

Aug 11, 2023

Yifeng-Sigma commented Aug 11, 2023 •

edited

Loading

Add a check to see type assertion fail or not.

Checklist

Code compiles correctly
Run make fmt to fix inconsistent formats
Run make lint to get lint errors and fix all of them
Created tests which fail without the change (if possible)
All tests passing
Extended the README / documentation, if necessary


          don't panic if type mismatch is decodearrow

b68a24b

Yifeng-Sigma had a problem deploying to gosnowflake-ci-env

August 11, 2023 21:20

— with

GitHub Actions Failure

Yifeng-Sigma requested review from a team and mtoader

August 11, 2023 21:21


          done

0ace8a7

Yifeng-Sigma temporarily deployed to gosnowflake-ci-env

August 11, 2023 21:38

— with

GitHub Actions Inactive

mtoader approved these changes

View reviewed changes

Yifeng-Sigma merged commit bc8b7b2 into master

3 of 4 checks passed

Yifeng-Sigma deleted the yifeng/no_panic branch

August 11, 2023 23:07

github-actions bot locked and limited conversation to collaborators

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.