aurora-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kevin Sweeney" <>
Subject Re: Review Request 36703: Replace Guava Joiner with String::join where simpler
Date Wed, 22 Jul 2015 22:08:32 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated July 22, 2015, 3:08 p.m.)

Review request for Aurora, Joshua Cohen and Bill Farner.


Review feedback.

Summary (updated)

Replace Guava Joiner with String::join where simpler

Repository: aurora

Description (updated)

Now that `String::join` is part of the standard library some uses of Guava's `Joiner` can
be removed.

Diffs (updated)

  src/main/java/org/apache/aurora/scheduler/base/ a5ffa5e95b301e536a84acf02817ea0c080559d0

  src/main/java/org/apache/aurora/scheduler/cron/ 904dd72dbb9036d8c06353a953bf35b8f04cfdbc

  src/main/java/org/apache/aurora/scheduler/cron/quartz/ 45e062d3acbd7b8565bd3773c1e994aae96378e0

  src/main/java/org/apache/aurora/scheduler/http/ e413ad9bdccc329777b3d0764ba8684539956679

  src/main/java/org/apache/aurora/scheduler/thrift/aop/ 7c8f8b9b8d7deb082edc0f85a6d3da1536735545



./gradlew -Pq build


Kevin Sweeney

  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message