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

feat: add support for map target type in Parquet options #1919

Merged
merged 7 commits into from
May 21, 2024

Conversation

sclmn
Copy link
Contributor

@sclmn sclmn commented May 15, 2024

The map target type creates a schema without the added key_value repeated field.

The map target type creates a schema without the added key_value repeated field.
@sclmn sclmn requested review from a team as code owners May 15, 2024 18:43
@sclmn sclmn requested a review from shollyman May 15, 2024 18:43
@product-auto-label product-auto-label bot added the size: s Pull request size is small. label May 15, 2024
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery API. label May 15, 2024
@Linchin
Copy link
Contributor

Linchin commented May 16, 2024

Thank you @sclmn for adding the new fields for us! We usually require 100% test coverage. Could you please also add unit tests for the new properties, similar to unit tests here? We also require the first commit and PR title to follow conventional commits. Could you follow the instruction in the link to update your PR? If you need any help with these, feel free to let us know :)

@Linchin Linchin assigned Linchin and unassigned chalmerlowe May 16, 2024
@sclmn sclmn changed the title Update format_options.py to include the newly added map target type. feat: add support for map target type in Parquet options May 17, 2024
Copy link

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@sclmn
Copy link
Contributor Author

sclmn commented May 17, 2024

I updated the PR. PTAL.

@Linchin Linchin added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@Linchin Linchin added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@Linchin Linchin added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@Linchin Linchin enabled auto-merge (squash) May 21, 2024 20:40
@Linchin Linchin added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label May 21, 2024
@Linchin Linchin merged commit c3f7b23 into googleapis:main May 21, 2024
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the googleapis/python-bigquery API. size: s Pull request size is small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants