Skip to content

Upgrade apache-arrow - #180

Merged
kravets-levko merged 1 commit into
mainfrom
upgrade-apache-arrow
Sep 18, 2023
Merged

Upgrade apache-arrow#180
kravets-levko merged 1 commit into
mainfrom
upgrade-apache-arrow

Conversation

@kravets-levko

Copy link
Copy Markdown
Contributor

Fixes#150

Signed-off-by: Levko Kravets <levko.ne@gmail.com>

@susodapopsusodapop left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does this break anything for older installations? If so, we will need to bump the major version number of this connector before we release it.

@kravets-levko

Copy link
Copy Markdown
ContributorAuthor

No it doesn't break anything except of actually fixing a linked bug. We have quite a lot of compatibility tests, and in addition I did a lot of manual testing. Also, we don't currently allow access to raw arrow batches like PySQL does, so from user perspective there should be no difference in library behavior

@kravets-levko
kravets-levko merged commit 4d46022 into mainSep 18, 2023
@kravets-levko
kravets-levko deleted the upgrade-apache-arrow branch September 18, 2023 15:38
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

scientific notation negative numbers

2 participants

@kravets-levko@susodapop