Skip to content

Handle formatOptions.useInt64Timestamp in tabledataListRequest#276

Merged
goccy merged 3 commits into
goccy:mainfrom
Recidiviz:dan/tabledata-list-int64
Apr 8, 2024
Merged

Handle formatOptions.useInt64Timestamp in tabledataListRequest#276
goccy merged 3 commits into
goccy:mainfrom
Recidiviz:dan/tabledata-list-int64

Conversation

@ohaibbq

@ohaibbq ohaibbq commented Mar 4, 2024

Copy link
Copy Markdown
Contributor

Closes #275

@goccy goccy left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Thank you for your contribution ! Please resolve conflict 🙏

@goccy goccy added the reviewed label Apr 6, 2024
@Gyllsdorff

Copy link
Copy Markdown

It looks like the only conflict is in go.mod and go.sum. Do you see any risks in doing a bump of all dependencies in this PR or are there any reasons to leave some deps on a older version?

This problem caused about 2 days of extra work until I saw what was wrong so it would be great if we could merge and release a fix.

ohaibbq PR is in a forked repo so I can't resolve the minor conflicts, I could fork-the-fork, resolve the conflict and create a new PR but that would be rude to Ohaibbq that did all the work. :)

@ohaibbq ohaibbq force-pushed the dan/tabledata-list-int64 branch from 974e0a7 to 55b8941 Compare April 8, 2024 06:19
@ohaibbq ohaibbq force-pushed the dan/tabledata-list-int64 branch from c4c0f28 to 7753753 Compare April 8, 2024 06:52
@ohaibbq

ohaibbq commented Apr 8, 2024

Copy link
Copy Markdown
Contributor Author

@goccy I've updated the dependencies and fixed a test failure

@goccy

goccy commented Apr 8, 2024

Copy link
Copy Markdown
Owner

LGTM 👍

@goccy goccy merged commit 0d38e32 into goccy:main Apr 8, 2024
iolalla pushed a commit to iolalla/bigquery-emulator that referenced this pull request Aug 16, 2025
…occy#276)

* Handle `formatOptions.useInt64Timestamp` in `tabledataListRequest`

* bump to 1.60.0; go mod tidy

* accept format options from request body
ohaibbq referenced this pull request in Recidiviz/bigquery-emulator Oct 29, 2025
…276)

* Handle `formatOptions.useInt64Timestamp` in `tabledataListRequest`

* bump to 1.60.0; go mod tidy

* accept format options from request body
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unmarshal fails to a struct with time.Time type when using the cloud.google.com/go/bigquery library

3 participants