Skip to content

Commit

Permalink
Use redis cluter (#72)
Browse files Browse the repository at this point in the history
* Change docker-compose.yml to redis cluster

* Adjust port mapping

* Git hub action update

* Jedis integration tests are using redis cluster

* Update connection string on lettuce

* Refactor Lettuce pooled abstraction to support cluster commands

* Refactor Lettuce pooled abstraction to support cluster commands - reformat

* Switch to LettuceUnified

* Switch to LettuceUnified and LettuceClusterPooled

* Switch tests to LettuceUnified

* Update Lettuce tests

* Update ports for integration-tests.yml

* Update cli command for integration-tests.yml

* Bump to 1.3.0

* Factory interface uses LettuceUnified

* Factory interface uses LettuceUnified - remove imports

* Update tests to accommodate possible delays

* bump to 1.3.1

* increase timeout wanting for test containers

* proper command health check command
  • Loading branch information
himadieievsv authored Oct 1, 2024
1 parent 806c8e8 commit 2bbad22
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 6 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/integration-tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ jobs:
env:
INITIAL_PORT: 7010
options: >-
--health-cmd "/redis-build/bin/redis-cli -p 7010 ping"
--health-cmd "redis-cli -p 7010 ping"
--health-interval 10s
--health-timeout 5s
--health-retries 5
Expand All @@ -31,7 +31,7 @@ jobs:
env:
INITIAL_PORT: 7020
options: >-
--health-cmd "/redis-build/bin/redis-cli -p 7020 ping"
--health-cmd "redis-cli -p 7020 ping"
--health-interval 10s
--health-timeout 5s
--health-retries 5
Expand All @@ -42,7 +42,7 @@ jobs:
env:
INITIAL_PORT: 7030
options: >-
--health-cmd "/redis-build/bin/redis-cli -p 7030 ping"
--health-cmd "redis-cli -p 7030 ping"
--health-interval 10s
--health-timeout 5s
--health-retries 5
Expand Down
2 changes: 1 addition & 1 deletion build.gradle.kts
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ plugins {

allprojects {
group = "com.himadieiev"
version = "1.3.0"
version = "1.3.1"

repositories {
mavenCentral()
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ class SemaphoreIntegrationTest {

(1..maxLeases)
.forEach {
assertTrue(semaphores[it - 1].lock("test", Duration.ofSeconds(1)))
assertTrue(semaphores[it - 1].lock("test", Duration.ofSeconds(2)))
}
val semaphores2 = mutableListOf<Semaphore>()
(1..maxLeases)
Expand All @@ -147,7 +147,7 @@ class SemaphoreIntegrationTest {
}

runBlocking {
delay(2000)
delay(3000)
}
(1..maxLeases)
.forEach {
Expand Down

0 comments on commit 2bbad22

Please sign in to comment.