Skip to content

Commit 4d00a79

Browse files
authored
deps: update veneer to 2.16.0 (#3855)
* deps: update veneer to 2.15.1 * deps: update veneer to 2.16.0 * fix deps, beam integration, and integration migration builds * cleanup * use variable * review feedback
1 parent 0dedbe2 commit 4d00a79

File tree

10 files changed

+84
-8
lines changed
  • bigtable-client-core-parent/bigtable-hbase-integration-tests-common
  • bigtable-dataflow-parent
  • bigtable-hbase-1.x-parent
  • bigtable-hbase-2.x-parent/bigtable-hbase-2.x
  • hbase-migration-tools/mirroring-client
    • bigtable-hbase-mirroring-client-1.x-parent/bigtable-hbase-mirroring-client-1.x-integration-tests
    • bigtable-hbase-mirroring-client-2.x-parent
      • bigtable-hbase-mirroring-client-1.x-2.x-integration-tests
      • bigtable-hbase-mirroring-client-2.x-integration-tests

10 files changed

+84
-8
lines changed

bigtable-client-core-parent/bigtable-hbase-integration-tests-common/pom.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,7 @@ limitations under the License.
6464
<artifactId>*</artifactId>
6565
</exclusion>
6666
</exclusions>
67+
<scope>test</scope>
6768
</dependency>
6869

6970
<dependency>

bigtable-dataflow-parent/bigtable-beam-import/pom.xml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,6 +59,7 @@ limitations under the License.
5959
<artifactId>gcs-connector</artifactId>
6060
<version>hadoop2-2.2.2</version>
6161
<classifier>shaded</classifier>
62+
<scope>runtime</scope>
6263
</dependency>
6364
<dependency>
6465
<groupId>org.apache.beam</groupId>
@@ -514,7 +515,7 @@ limitations under the License.
514515
<plugin>
515516
<groupId>org.apache.maven.plugins</groupId>
516517
<artifactId>maven-failsafe-plugin</artifactId>
517-
<version>3.0.0-M4</version>
518+
<version>3.0.0-M7</version>
518519
<executions>
519520
<execution>
520521
<id>beam-integration-test</id>

bigtable-dataflow-parent/bigtable-hbase-beam/pom.xml

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -71,6 +71,11 @@ limitations under the License.
7171
<groupId>org.hamcrest</groupId>
7272
<artifactId>hamcrest</artifactId>
7373
</exclusion>
74+
<!-- we set our own version of hadoop, excluding to avoid dependency conflicts-->
75+
<exclusion>
76+
<groupId>org.apache.hadoop</groupId>
77+
<artifactId>*</artifactId>
78+
</exclusion>
7479
</exclusions>
7580
</dependency>
7681

@@ -326,6 +331,16 @@ limitations under the License.
326331
</execution>
327332
</executions>
328333
</plugin>
334+
<plugin>
335+
<groupId>org.apache.maven.plugins</groupId>
336+
<artifactId>maven-dependency-plugin</artifactId>
337+
<configuration>
338+
<ignoredNonTestScopedDependencies>
339+
<!-- required per verify-mirror-deps-->
340+
<ignoredNonTestScopedDependency>com.google.code.findbugs:jsr305:*</ignoredNonTestScopedDependency>
341+
</ignoredNonTestScopedDependencies>
342+
</configuration>
343+
</plugin>
329344
</plugins>
330345
</build>
331346
</project>

bigtable-hbase-1.x-parent/bigtable-hbase-1.x-tools/pom.xml

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -62,11 +62,6 @@
6262
</exclusion>
6363
</exclusions>
6464
</dependency>
65-
<dependency>
66-
<groupId>com.google.guava</groupId>
67-
<artifactId>guava</artifactId>
68-
<version>${guava.version}</version>
69-
</dependency>
7065
<dependency>
7166
<groupId>com.google.code.findbugs</groupId>
7267
<artifactId>jsr305</artifactId>
@@ -89,6 +84,13 @@
8984
<version>1.2.11</version>
9085
<scope>runtime</scope>
9186
</dependency>
87+
<dependency>
88+
<groupId>com.google.guava</groupId>
89+
<artifactId>guava</artifactId>
90+
<version>${guava.version}</version>
91+
<!-- todo: align guava deps - we use repackaged guava in the main classes-->
92+
<scope>test</scope>
93+
</dependency>
9294
<dependency>
9395
<groupId>org.mockito</groupId>
9496
<artifactId>mockito-core</artifactId>

bigtable-hbase-1.x-parent/bigtable-hbase-1.x/pom.xml

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -202,4 +202,27 @@ limitations under the License.
202202
<scope>test</scope>
203203
</dependency>
204204
</dependencies>
205+
206+
<build>
207+
<plugins>
208+
<plugin>
209+
<groupId>org.apache.maven.plugins</groupId>
210+
<artifactId>maven-dependency-plugin</artifactId>
211+
<version>3.3.0</version>
212+
<configuration>
213+
<ignoredNonTestScopedDependencies>
214+
<!-- Temporarily disable dependency plugin until we figure out why it thinks compile deps like google-cloud-bigtable are unused
215+
For example:
216+
[WARNING] Non-test scoped test only dependencies found:
217+
[WARNING] com.google.cloud:google-cloud-bigtable:jar:2.16.0:compile
218+
...
219+
220+
Same problem exists in bigtable-hbase-2.x
221+
-->
222+
<ignoredNonTestScopedDependency>*</ignoredNonTestScopedDependency>
223+
</ignoredNonTestScopedDependencies>
224+
</configuration>
225+
</plugin>
226+
</plugins>
227+
</build>
205228
</project>

bigtable-hbase-2.x-parent/bigtable-hbase-2.x/pom.xml

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -185,6 +185,18 @@ limitations under the License.
185185
</dependencies>
186186
<build>
187187
<plugins>
188+
<plugin>
189+
<groupId>org.apache.maven.plugins</groupId>
190+
<artifactId>maven-dependency-plugin</artifactId>
191+
<version>3.3.0</version>
192+
<configuration>
193+
<ignoredNonTestScopedDependencies>
194+
<!-- Temporarily disable dependency plugin until we figure out why it thinks compile deps like google-cloud-bigtable are unused
195+
See bigtable-hbase-1.x pom.xml for more details-->
196+
<ignoredNonTestScopedDependency>*</ignoredNonTestScopedDependency>
197+
</ignoredNonTestScopedDependencies>
198+
</configuration>
199+
</plugin>
188200
<plugin>
189201
<!-- skip for bigtable-hbase to allow for upcoming refactoring-->
190202
<groupId>org.codehaus.mojo</groupId>

hbase-migration-tools/mirroring-client/bigtable-hbase-mirroring-client-1.x-parent/bigtable-hbase-mirroring-client-1.x-integration-tests/pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -295,6 +295,13 @@ limitations under the License.
295295
<version>1.9.1</version>
296296
<scope>test</scope>
297297
</dependency>
298+
<dependency>
299+
<groupId>org.junit.vintage</groupId>
300+
<artifactId>junit-vintage-engine</artifactId>
301+
<version>${junit-vintage-engine.version}</version>
302+
<scope>test</scope>
303+
</dependency>
304+
298305
<dependency>
299306
<groupId>com.google.truth</groupId>
300307
<artifactId>truth</artifactId>

hbase-migration-tools/mirroring-client/bigtable-hbase-mirroring-client-2.x-parent/bigtable-hbase-mirroring-client-1.x-2.x-integration-tests/pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -280,6 +280,13 @@ limitations under the License.
280280
<scope>test</scope>
281281
</dependency>
282282

283+
<dependency>
284+
<groupId>org.junit.vintage</groupId>
285+
<artifactId>junit-vintage-engine</artifactId>
286+
<version>${junit-vintage-engine.version}</version>
287+
<scope>test</scope>
288+
</dependency>
289+
283290
<dependency>
284291
<groupId>org.junit.platform</groupId>
285292
<artifactId>junit-platform-launcher</artifactId>

hbase-migration-tools/mirroring-client/bigtable-hbase-mirroring-client-2.x-parent/bigtable-hbase-mirroring-client-2.x-integration-tests/pom.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -285,6 +285,13 @@ limitations under the License.
285285
<scope>test</scope>
286286
</dependency>
287287

288+
<dependency>
289+
<groupId>org.junit.vintage</groupId>
290+
<artifactId>junit-vintage-engine</artifactId>
291+
<version>${junit-vintage-engine.version}</version>
292+
<scope>test</scope>
293+
</dependency>
294+
288295
<dependency>
289296
<groupId>org.junit.platform</groupId>
290297
<artifactId>junit-platform-launcher</artifactId>

pom.xml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ limitations under the License.
3333
<parent>
3434
<groupId>com.google.cloud</groupId>
3535
<artifactId>google-cloud-shared-config</artifactId>
36-
<version>1.3.2</version>
36+
<version>1.5.4</version>
3737
</parent>
3838

3939
<licenses>
@@ -56,7 +56,7 @@ limitations under the License.
5656
<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
5757

5858
<!-- core dependency versions -->
59-
<bigtable.version>2.14.1</bigtable.version>
59+
<bigtable.version>2.16.0</bigtable.version>
6060
<bigtable-client-core.version>1.27.1</bigtable-client-core.version>
6161
<grpc-conscrypt.version>2.5.2</grpc-conscrypt.version>
6262
<!-- keeping at this version to align with hbase-->
@@ -76,6 +76,7 @@ limitations under the License.
7676
<!-- testing dependency versions -->
7777
<commons-lang.version>2.6</commons-lang.version>
7878
<junit.version>4.13.2</junit.version>
79+
<junit-vintage-engine.version>5.9.1</junit-vintage-engine.version>
7980
<hamcrest.version>1.3</hamcrest.version>
8081
<mockito.version>4.9.0</mockito.version>
8182
<!-- TODO: check if commons-codec was transitively updated to 1.13 and okhttp was updated to 2.7.5 when upgrading-->

0 commit comments

Comments
 (0)