Skip to content

Commit

Permalink
Destination Bigquery: Remove reference to TDValve (#37714)
Browse files Browse the repository at this point in the history
  • Loading branch information
edgao committed May 8, 2024
1 parent 16ed6bf commit 47c86a9
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 7 deletions.
Expand Up @@ -3,15 +3,15 @@ plugins {
}

airbyteJavaConnector {
cdkVersionRequired = '0.33.2'
cdkVersionRequired = '0.34.0'
features = [
'db-destinations',
'datastore-bigquery',
'typing-deduping',
'gcs-destinations',
'core',
]
useLocalCdk = false
useLocalCdk = true
}

java {
Expand Down
Expand Up @@ -5,7 +5,7 @@ data:
connectorSubtype: database
connectorType: destination
definitionId: 22f6c74f-5699-40ff-833c-4a879ea40133
dockerImageTag: 2.4.15
dockerImageTag: 2.4.16
dockerRepository: airbyte/destination-bigquery
documentationUrl: https://docs.airbyte.com/integrations/destinations/bigquery
githubIssueLabel: destination-bigquery
Expand Down
Expand Up @@ -11,7 +11,6 @@
import io.airbyte.cdk.integrations.util.ConnectorExceptionUtil;
import io.airbyte.integrations.base.destination.typing_deduping.ParsedCatalog;
import io.airbyte.integrations.base.destination.typing_deduping.StreamConfig;
import io.airbyte.integrations.base.destination.typing_deduping.TypeAndDedupeOperationValve;
import io.airbyte.integrations.base.destination.typing_deduping.TyperDeduper;
import io.airbyte.integrations.destination.bigquery.formatter.DefaultBigQueryRecordFormatter;
import io.airbyte.integrations.destination.bigquery.uploader.AbstractBigQueryUploader;
Expand Down Expand Up @@ -41,7 +40,6 @@ class BigQueryRecordConsumer extends FailureTrackingAirbyteMessageConsumer imple
private final String defaultDatasetId;
private AirbyteMessage lastStateMessage = null;

private final TypeAndDedupeOperationValve streamTDValve = new TypeAndDedupeOperationValve();
private final ParsedCatalog catalog;
private final TyperDeduper typerDeduper;

Expand Down Expand Up @@ -125,7 +123,7 @@ public void close(final boolean hasFailed) throws Exception {
uploaderMap.forEach((streamId, uploader) -> {
try {
uploader.close(hasFailed, outputRecordCollector, lastStateMessage);
typerDeduper.typeAndDedupe(streamId.getNamespace(), streamId.getName(), true);
typerDeduper.typeAndDedupe(streamId.getNamespace(), streamId.getName());
} catch (final Exception e) {
exceptionsThrown.add(e);
LOGGER.error("Exception while closing uploader {}", uploader, e);
Expand Down
3 changes: 2 additions & 1 deletion docs/integrations/destinations/bigquery.md
Expand Up @@ -220,7 +220,8 @@ tutorials:

| Version | Date | Pull Request | Subject |
| :------ | :--------- | :--------------------------------------------------------- | :-------------------------------------------------------------------------------------------------------------------------------------------------------------- |
| 2.4.15 | 2024-05-07 | [34611](https://github.com/airbytehq/airbyte/pull/34611) | Adopt CDK 0.33.2 |
| 2.4.16 | 2024-05-08 | [37714](https://github.com/airbytehq/airbyte/pull/37714) | Adopt CDK 0.34.0 |
| 2.4.15 | 2024-05-07 | [34611](https://github.com/airbytehq/airbyte/pull/34611) | Adopt CDK 0.33.2 |
| 2.4.14 | 2024-02-25 | [37584](https://github.com/airbytehq/airbyte/pull/37584) | Remove unused insecure dependencies from CDK |
| 2.4.13 | 2024-02-25 | [36899](https://github.com/airbytehq/airbyte/pull/36899) | adopt latest CDK |
| 2.4.12 | 2024-03-04 | [35315](https://github.com/airbytehq/airbyte/pull/35315) | Adopt CDK 0.23.11 |
Expand Down

0 comments on commit 47c86a9

Please sign in to comment.