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
Modified parameters for Dataflow templates #1502
Modified parameters for Dataflow templates #1502
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1502 +/- ##
=========================================
Coverage 40.93% 40.93%
- Complexity 2808 2809 +1
=========================================
Files 739 739
Lines 42855 42855
Branches 4582 4582
=========================================
+ Hits 17542 17543 +1
Misses 23814 23814
+ Partials 1499 1498 -1
|
v1/src/main/java/com/google/cloud/teleport/templates/CSVToBigQuery.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/PubsubToAvro.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/PubsubToAvro.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/PubsubToAvro.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/PubsubToAvro.java
Outdated
Show resolved
Hide resolved
...c-to-googlecloud/src/main/java/com/google/cloud/teleport/v2/options/JdbcToPubsubOptions.java
Outdated
Show resolved
Hide resolved
...c-to-googlecloud/src/main/java/com/google/cloud/teleport/v2/options/JdbcToPubsubOptions.java
Outdated
Show resolved
Hide resolved
v2/kafka-to-bigquery/src/main/java/com/google/cloud/teleport/v2/templates/KafkaToBigQuery.java
Outdated
Show resolved
Hide resolved
...oud/src/main/java/com/google/cloud/teleport/v2/mongodb/options/MongoDbToBigQueryOptions.java
Outdated
Show resolved
Hide resolved
v2/pubsub-to-redis/src/main/java/com/google/cloud/teleport/v2/templates/PubSubToRedis.java
Outdated
Show resolved
Hide resolved
Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
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.
Just a few minor updates. Once those are complete, looks good to me.
v1/src/main/java/com/google/cloud/teleport/bigtable/BigtableToJson.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/common/SpannerConverters.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/common/SpannerConverters.java
Outdated
Show resolved
Hide resolved
v1/src/main/java/com/google/cloud/teleport/templates/common/SpannerConverters.java
Outdated
Show resolved
Hide resolved
v2/common/src/main/java/com/google/cloud/teleport/v2/transforms/BigQueryConverters.java
Outdated
Show resolved
Hide resolved
v2/common/src/main/java/com/google/cloud/teleport/v2/transforms/BigQueryConverters.java
Outdated
Show resolved
Hide resolved
v2/common/src/main/java/com/google/cloud/teleport/v2/transforms/BigQueryConverters.java
Outdated
Show resolved
Hide resolved
.../main/java/com/google/cloud/teleport/v2/elasticsearch/options/ElasticsearchWriteOptions.java
Outdated
Show resolved
Hide resolved
v2/pubsub-to-redis/src/main/java/com/google/cloud/teleport/v2/templates/PubSubToRedis.java
Outdated
Show resolved
Hide resolved
…Json.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…annerConverters.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…s/BigQueryConverters.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…s/BigQueryConverters.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…s/BigQueryConverters.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…t/v2/elasticsearch/options/ElasticsearchWriteOptions.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…templates/PubSubToRedis.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…annerConverters.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
…annerConverters.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
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.
One typo fix, otherwise LGTM
.../main/java/com/google/cloud/teleport/v2/elasticsearch/options/ElasticsearchWriteOptions.java
Outdated
Show resolved
Hide resolved
…t/v2/elasticsearch/options/ElasticsearchWriteOptions.java Co-authored-by: Rebecca Szper <98840847+rszper@users.noreply.github.com>
1acbde8
into
GoogleCloudPlatform:main
No description provided.