-
Notifications
You must be signed in to change notification settings - Fork 85
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
PLUGIN-1438: Added support for destination table write preference in BQ Execute Plugin #1473
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add evidence/screenshots in PR description under Tested
section.
A good PR description example: #1444 (comment)
@Description("Specifies the write preference for the destination table. Options are: " + | ||
"Write if Empty: Only write if the table is empty. " + | ||
"Append to Table: Add results to existing data. " + | ||
"Overwrite Table: Replace all existing data.") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Description/Tooltip should be crisp: Specifies whether a job should overwrite or append the existing destination table if it already exists.
Please add these details in markdown: https://github.com/data-integrations/google-cloud/blob/develop/docs/BigQueryExecute-action.md
src/main/java/io/cdap/plugin/gcp/bigquery/action/BigQueryExecute.java
Outdated
Show resolved
Hide resolved
src/main/java/io/cdap/plugin/gcp/bigquery/action/BigQueryExecute.java
Outdated
Show resolved
Hide resolved
src/test/java/io/cdap/plugin/gcp/bigquery/action/BigQueryExecuteTest.java
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, please squash commits before merge.
40b5f74
to
d7b5253
Compare
…ugin Addressed PR comments: updated description, added documentation for the added feature Formatted to fix failing tests Refactor: Replace hardcoded write preference strings with JobInfo.WriteDisposition constants Refactor: Simplify error message for invalid write preference validation Refactor: Centralize valid write preferences in BigQueryExecute.Config Refactor: Dynamically generate valid write preferences from JobInfo.WriteDisposition
d7b5253
to
a467a7a
Compare
Added support for destination table write preference in BQ Execute Plugin
Jira : PLUGIN-1438
Description
This update introduces the ability for users to select a destination table write preference when storing results in BigQuery. Users can now choose from the following options:
UI Field
BigQueryExecute-action.json
Docs
BigQueryExecute-action.md
Code change
BigQueryExecute.java
Unit Tests
BigQueryExecuteTest.java
Tested