Skip to content

Commit 3ebde31

Browse files
committed
TRR: Dependency remediation (WiP)
1 parent 71d8139 commit 3ebde31

5 files changed

Lines changed: 14 additions & 19 deletions

File tree

plugin/virtualized-table-plugin/pom.xml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -51,13 +51,17 @@
5151
<version>${scala.version}</version>
5252
</dependency>
5353

54+
<dependency>
55+
<groupId>com.github.ben-manes.caffeine</groupId>
56+
<artifactId>caffeine</artifactId>
57+
<version>${caffeine.version}</version>
58+
</dependency>
59+
5460
<dependency>
5561
<groupId>junit</groupId>
5662
<artifactId>junit</artifactId>
5763
</dependency>
5864

59-
60-
6165
<dependency>
6266
<groupId>org.scalatest</groupId>
6367
<artifactId>scalatest_2.13</artifactId>

plugin/virtualized-table-plugin/src/main/scala/org/finos/vuu/plugin/virtualized/table/WindowedCache.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package org.finos.vuu.plugin.virtualized.table
22

33
import org.finos.toolbox.time.Clock
44
import org.finos.vuu.core.table.RowData
5-
import org.finos.vuu.plugin.virtualized.table.cache.GuavaWindowedRowDataCache
5+
import org.finos.vuu.plugin.virtualized.table.cache.CaffeineWindowedRowDataCache
66

77
trait WindowedCache[KEY, VALUE] {
88
def put(key: KEY, v: VALUE): Unit
@@ -13,7 +13,7 @@ trait WindowedCache[KEY, VALUE] {
1313

1414
object RowDataCache{
1515
def apply(cacheSize: Int)(implicit clock: Clock): WindowedCache[String, RowData] = {
16-
new GuavaWindowedRowDataCache(cacheSize)
16+
new CaffeineWindowedRowDataCache(cacheSize)
1717
}
1818

1919
}

plugin/virtualized-table-plugin/src/main/scala/org/finos/vuu/plugin/virtualized/table/cache/GuavaWindowedRowDataCache.scala renamed to plugin/virtualized-table-plugin/src/main/scala/org/finos/vuu/plugin/virtualized/table/cache/CaffeineWindowedRowDataCache.scala

Lines changed: 5 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package org.finos.vuu.plugin.virtualized.table.cache
22

3-
import com.google.common.cache.{Cache, CacheBuilder, RemovalListener, RemovalNotification}
3+
import com.github.benmanes.caffeine.cache.{Cache, Caffeine, RemovalCause}
44
import com.typesafe.scalalogging.StrictLogging
55
import org.finos.toolbox.logging.LogAtFrequency
66
import org.finos.toolbox.time.Clock
@@ -9,22 +9,19 @@ import org.finos.vuu.plugin.virtualized.table.WindowedCache
99

1010
import java.util.concurrent.atomic.AtomicInteger
1111

12-
class GuavaWindowedRowDataCache(val cacheSize: Int)(implicit clock: Clock) extends WindowedCache[String, RowData] with StrictLogging {
12+
class CaffeineWindowedRowDataCache(val cacheSize: Int)(implicit clock: Clock) extends WindowedCache[String, RowData] with StrictLogging {
1313

1414
val logAtFrequency = new LogAtFrequency(10_000)
1515
val removalCounter = new AtomicInteger()
1616

17-
val cache: Cache[String, RowData] = CacheBuilder.newBuilder()
17+
val cache: Cache[String, RowData] = Caffeine.newBuilder()
1818
.maximumSize(cacheSize)
19-
.removalListener((notification: RemovalNotification[String, RowData]) => {
20-
19+
.removalListener((_: String, _: RowData, _: RemovalCause) => {
2120
val count = removalCounter.incrementAndGet()
22-
23-
if(logAtFrequency.shouldLog()){
21+
if (logAtFrequency.shouldLog()) {
2422
logger.debug(s"[ROWCACHE] Removing ${count} rowCache keys in last 10 seconds")
2523
removalCounter.set(0)
2624
}
27-
2825
})
2926
.build()
3027

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,7 @@
7575
<maven.compiler.source>11</maven.compiler.source>
7676
<maven.compiler.target>11</maven.compiler.target>
7777
<scala.version>2.13.14</scala.version>
78+
<caffeine.version>3.2.1</caffeine.version>
7879
<metrics.version>4.2.5</metrics.version>
7980
<curator.version>3.1.0</curator.version>
8081
<scalatest.version>3.2.19</scalatest.version>
@@ -90,7 +91,6 @@
9091
<logback.version>1.5.6</logback.version>
9192
<slf4j.version>2.0.13</slf4j.version>
9293
<jmh.version>1.36</jmh.version>
93-
<guava.version>32.0.1-jre</guava.version>
9494
<maven.gpg.plugin.version>3.1.0</maven.gpg.plugin.version>
9595
<gpg.executable>gpg</gpg.executable>
9696
<publish.server.id>ossrh</publish.server.id>

vuu/pom.xml

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -20,12 +20,6 @@
2020
<version>0.9.76-SNAPSHOT</version>
2121
</dependency>
2222

23-
<dependency>
24-
<groupId>com.google.guava</groupId>
25-
<artifactId>guava</artifactId>
26-
<version>${guava.version}</version>
27-
</dependency>
28-
2923
<dependency>
3024
<groupId>org.scala-lang</groupId>
3125
<artifactId>scala-library</artifactId>

0 commit comments

Comments
 (0)