Skip to content
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.

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

fix: Add DateDayVector support #141

Merged
merged 5 commits into from
Nov 26, 2023

Conversation

spieden
Copy link
Contributor

@spieden spieden commented Nov 23, 2023

@candiduslynx
Copy link
Contributor

@spieden thanks for the contribution!

Could you also add the code to test date types to https://github.com/cloudquery/plugin-sdk-java/blob/main/lib/src/test/java/io/cloudquery/helper/ArrowHelperTest.java (or to the same test package)?

Copy link
Contributor

@candiduslynx candiduslynx left a comment

Choose a reason for hiding this comment

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

Looks good, but I'd love to also see tests added for this

@spieden
Copy link
Contributor Author

spieden commented Nov 25, 2023

@candiduslynx Ok done. I removed the DateMilliVector support because it was failing round trips and I'm not using it anyway. (Values go in as longs and come out as some kind of DateTime.)

@candiduslynx
Copy link
Contributor

@spieden the build for the ci workflow fails

@spieden
Copy link
Contributor Author

spieden commented Nov 26, 2023

@candiduslynx Hmm, fixed locally but not getting a status back 🤔

@candiduslynx candiduslynx added the automerge Add to automerge PRs once requirements are met label Nov 26, 2023
@kodiakhq kodiakhq bot merged commit 21e4981 into cloudquery:main Nov 26, 2023
4 checks passed
@candiduslynx candiduslynx changed the title fix: add date types to ArrowHelper fix: Add DateDayVector support Nov 26, 2023
kodiakhq bot pushed a commit that referenced this pull request Nov 27, 2023
🤖 I have created a release *beep* *boop*
---


## [0.0.13](v0.0.12...v0.0.13) (2023-11-26)


### Bug Fixes

* add date types to ArrowHelper ([#141](#141)) ([21e4981](21e4981))
* **deps:** Update dependency com.fasterxml.jackson.core:jackson-annotations to v2.15.3 ([#133](#133)) ([627b49d](627b49d))
* **deps:** Update dependency com.fasterxml.jackson.core:jackson-core to v2.15.3 ([#135](#135)) ([235781d](235781d))
* **deps:** Update dependency com.google.guava:guava to v32.1.3-jre ([#136](#136)) ([eb7b853](eb7b853))
* **deps:** Update dependency gradle to v8.4 ([#137](#137)) ([8675852](8675852))
* **deps:** Update dependency io.grpc:grpc-protobuf to v1.59.0 ([#138](#138)) ([88d3e71](88d3e71))
* **deps:** Update dependency io.grpc:grpc-stub to v1.59.0 ([#140](#140)) ([556b91b](556b91b))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
kodiakhq bot pushed a commit that referenced this pull request Nov 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Add to automerge PRs once requirements are met
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants