Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

OSS Sync | Sun Sep 03 23:29:17 UTC 2023 #567

Open
wants to merge 52 commits into
base: master
Choose a base branch
from
Open

OSS Sync | Sun Sep 03 23:29:17 UTC 2023 #567

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
52 commits
Select commit Hold shift + click to select a range
14fde53
Ds scalafmt (#32485)
leonbur Feb 6, 2023
08a62da
Remove unused test deps (#33323)
Mar 10, 2023
66f88e5
Blocking retries attempts tracking fix (#32757)
atamurius Mar 14, 2023
2a49a32
Revert "Blocking retries attempts tracking fix" (#33818)
natansil Apr 3, 2023
87d8c14
[greyhound] remove internal topic creation - wix adapter (#33820)
berman7 Apr 3, 2023
199c66c
[greyhound] parallel consumer OffsetsAndGaps (#33605)
ben-wattelman Apr 4, 2023
b46ad51
Blocking retries attempts tracking fix + fix transitive header bug (#…
natansil Apr 4, 2023
693ab11
[greyhound] expose internal kafka producer metrics (#34160)
berman7 Apr 22, 2023
659129c
[greyhound] Cooperative Rebalance fix (#34153)
leonbur Apr 23, 2023
ae1bb8f
code cleanup (#34171)
leonbur Apr 23, 2023
57d215b
[greyhound] parallel consumer implementation (#34061)
ben-wattelman May 24, 2023
b08f019
[gh-consumers-proxy] s3 bridge (#34839)
berman7 May 29, 2023
bb658a6
[greyhound] parallel consumer - add visibility (#34908)
ben-wattelman May 30, 2023
1222ad9
[greyhound] parallel consumer visibility (#34926)
ben-wattelman Jun 1, 2023
85381bf
[greyhound] parallel consumer - add offsets and gaps init (#35027)
ben-wattelman Jun 6, 2023
dd1aee1
[greyhound] parallel consumer - add grouping for no-key records (#35071)
ben-wattelman Jun 11, 2023
cab79d4
make GreyhoundNG RetryConfig serializable (so can be sent over gRpc a…
natansil Jun 13, 2023
a6d1ebe
multi-tenant consumer proxy redesign - initial commit (#35244)
berman7 Jun 17, 2023
ad13e68
[greyhound] parallel consumer - add gaps limit (#35313)
ben-wattelman Jun 20, 2023
6e2186a
Fixes in preparation for ZIO 2.0.15 (#35320)
leonbur Jun 21, 2023
5561154
[greyhound-consumer-proxy] start from latest offset when group doesn'…
berman7 Jun 22, 2023
1186147
[greyhound] parallel consumer - compression and encoding for gaps lim…
ben-wattelman Jun 25, 2023
e1373a5
greyhound proxy non blocking retries (#35456)
natansil Jun 29, 2023
797de70
new kafka monitor server for proxy (currently) (#35598)
berman7 Jul 2, 2023
d7f277d
[greyhound] fix OffsetsInitializer metadata bug (#35684)
ben-wattelman Jul 6, 2023
975c135
ConsumerIT: add a test: allow to override offsetReset with autoResetO…
natansil Jul 13, 2023
b69b93b
gh RecordConsumer - add visibility to extra properties setup with aut…
natansil Jul 13, 2023
b3e9658
gh RecordConsumer - allow to override offsetReset with autoResetOffse…
natansil Jul 16, 2023
c19bee0
[greyhound] gaps limit calculation script (#35898)
ben-wattelman Jul 16, 2023
ee3e950
[greyhound] parallel consumer - update gaps limit (#35902)
ben-wattelman Jul 16, 2023
03122e3
[gh-consumer-proxy] fix: if position is lower than beginning offset -…
berman7 Jul 16, 2023
7434dd7
fix build for greyhound publish to maven (#35951)
natansil Jul 18, 2023
cb30bb4
[greyhound] parallel consumer - improve logging (#36009)
ben-wattelman Jul 20, 2023
9b676c9
[greyhound] parallel consumer - add logs (#36082)
ben-wattelman Jul 24, 2023
118de10
[greyhound] parallel consumer - fix update bug (#36161)
ben-wattelman Jul 26, 2023
e57a8bd
[greyhound] change onPartitionsAssigned result to DelayedRebalanceEff…
ben-wattelman Jul 27, 2023
00d10db
fix batch consumer test timeout interruption (#36246)
leonbur Jul 30, 2023
e2fdcb5
[greyhound] parallel consumer - fix skip logic (#36281)
ben-wattelman Jul 31, 2023
d8420ab
[greyhound] parallel consumer - improve visibility (#36346)
ben-wattelman Aug 3, 2023
3af93b9
[greyhound] parallel consumer - init gaps on every rebalance (#36375)
ben-wattelman Aug 9, 2023
2abfa3b
[greyhound] parallel cosnumer - add init log (#36421)
ben-wattelman Aug 9, 2023
a27d84e
[greyhound] parallel consumer - fix shutdown (#36489)
ben-wattelman Aug 15, 2023
f61ce82
[greyhound] add shutdown visibility (#36571)
ben-wattelman Aug 16, 2023
94e8131
[greyhound] parallel consumer - interrupt fiber on shutdown timeout (…
ben-wattelman Aug 20, 2023
9468545
[greyhound] parallel consumer - add disconnect on shutdown (#36660)
ben-wattelman Aug 21, 2023
461ccba
[greyhound] parallel consumer - add OffsetsAndGaps visibility (#36685)
ben-wattelman Aug 22, 2023
41ade44
enrich commit metadata with pod and timestamp (#36679)
leonbur Aug 23, 2023
785d0b9
Metadata to commit log (#36749)
leonbur Aug 27, 2023
c9d6404
GH commit metadata ts is in UTC (#36774)
leonbur Aug 28, 2023
cee1cac
moved enrich metadata out of GH core. add it to commit on rebalance (…
leonbur Aug 28, 2023
10ffeb2
[greyhound] parallel consumer - remove noisy logs (#36777)
ben-wattelman Aug 28, 2023
d723223
[consumer-proxy] fix retry bug (#36902)
berman7 Sep 3, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions core/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,7 @@ scala_library(
srcs = [
"//core/src/main/scala/com/wixpress/dst/greyhound/core:sources",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/admin:sources",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/compression:sources",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/consumer:sources",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/consumer/batched:sources",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/consumer/domain:sources",
Expand All @@ -26,6 +27,7 @@ scala_library(
"@dev_zio_zio_managed_2_12",
"@dev_zio_zio_stacktracer_2_12",
"@dev_zio_zio_streams_2_12",
"@org_apache_commons_commons_compress",
"@org_apache_kafka_kafka_clients",
"@org_slf4j_slf4j_api",
],
Expand Down
6 changes: 3 additions & 3 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/AdminClientIT.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ import com.wixpress.dst.greyhound.core.producer.ProducerRecord
import com.wixpress.dst.greyhound.core.testkit.{BaseTestWithSharedEnv, TestMetrics}
import com.wixpress.dst.greyhound.core.zioutils.CountDownLatch
import com.wixpress.dst.greyhound.testenv.ITEnv
import com.wixpress.dst.greyhound.testenv.ITEnv.{Env, TestResources, testResources}
import com.wixpress.dst.greyhound.testenv.ITEnv.{testResources, Env, TestResources}
import org.apache.kafka.common.config.TopicConfig.{DELETE_RETENTION_MS_CONFIG, MAX_MESSAGE_BYTES_CONFIG, RETENTION_MS_CONFIG}
import org.apache.kafka.common.errors.InvalidTopicException
import org.specs2.specification.core.Fragments
Expand Down Expand Up @@ -83,7 +83,7 @@ class AdminClientIT extends BaseTestWithSharedEnv[Env, TestResources] {
}
}

//todo uncomment this after https://github.com/wix-private/core-server-build-tools/pull/13043 is merged
// todo uncomment this after https://github.com/wix-private/core-server-build-tools/pull/13043 is merged
// "reflect errors" in {
// val topic1 = aTopicConfig()
// val topic2 = aTopicConfig("x" * 250)
Expand All @@ -104,7 +104,7 @@ class AdminClientIT extends BaseTestWithSharedEnv[Env, TestResources] {
// created === Map(badTopic.name -> None)
// }
// }
//todo uncomment this after https://github.com/wix-private/core-server-build-tools/pull/13043 is merged
// todo uncomment this after https://github.com/wix-private/core-server-build-tools/pull/13043 is merged
// =================================================================================================================================
"ignore TopicExistsException by default" in {
val topic = aTopicConfig()
Expand Down
5 changes: 0 additions & 5 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,13 +24,8 @@ specs2_ite2e_test(
"//core/src/main/scala/com/wixpress/dst/greyhound/core/producer",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/zioutils",
"//core/src/test/resources",
#"//core/src/test/scala/com/wixpress/dst/greyhound/core/consumer",
#"//core/src/test/scala/com/wixpress/dst/greyhound/core/testkit",
"@ch_qos_logback_logback_classic",
# "@dev_zio_izumi_reflect_2_12",
"@dev_zio_zio_2_12",
"@dev_zio_zio_test_2_12",
"@org_apache_kafka_kafka_2_12",
"@org_apache_kafka_kafka_clients",
"//core/src/test/scala/com/wixpress/dst/greyhound/core/testkit",
],
Expand Down
957 changes: 564 additions & 393 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/ConsumerIT.scala
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 1 addition & 0 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/batched/BatchedConsumerIT.scala
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -212,6 +212,7 @@ class BatchedConsumerIT extends BaseTestWithSharedEnv[Env, TestResources] {
) *>
barrier
.offer(())
.interruptible
// we can't block here, otherwise rebalance won't happen - so we just fail
.timeoutFail(new RuntimeException("queue full"))(1.second)
.tapError(_ => ZIO.succeed(println(s"[$id] timed out waiting on barrier")))
Expand Down
3 changes: 0 additions & 3 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/offset/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ specs2_ite2e_test(
"@dev_zio_izumi_reflect_2_12",
"@dev_zio_zio_managed_2_12",
"//core/src/it/resources",
"//core/src/it/scala/com/wixpress/dst/greyhound/core",
"//core/src/it/scala/com/wixpress/dst/greyhound/testenv",
"//core/src/it/scala/com/wixpress/dst/greyhound/testkit",
"//core/src/main/scala/com/wixpress/dst/greyhound/core",
Expand All @@ -23,9 +22,7 @@ specs2_ite2e_test(
"//core/src/main/scala/com/wixpress/dst/greyhound/core/metrics",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/producer",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/zioutils",
"//core/src/test/scala/com/wixpress/dst/greyhound/core/consumer",
"//core/src/test/scala/com/wixpress/dst/greyhound/core/testkit",
"@ch_qos_logback_logback_classic",
# "@dev_zio_izumi_reflect_2_12",
"@dev_zio_zio_2_12",
"@org_apache_kafka_kafka_clients",
Expand Down
27 changes: 27 additions & 0 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/parallel/BUILD.bazel
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,27 @@
package(default_visibility = ["//visibility:public"])

sources()

specs2_ite2e_test(
name = "parallel",
srcs = [
":sources",
],
deps = [
"//core/src/it/resources",
"//core/src/it/scala/com/wixpress/dst/greyhound/testenv",
"//core/src/it/scala/com/wixpress/dst/greyhound/testkit",
"//core/src/main/scala/com/wixpress/dst/greyhound/core",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/consumer",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/consumer/domain",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/consumer/retry",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/metrics",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/producer",
"//core/src/main/scala/com/wixpress/dst/greyhound/core/zioutils",
"//core/src/test/scala/com/wixpress/dst/greyhound/core/testkit",
"@dev_zio_izumi_reflect_2_12",
"@dev_zio_zio_2_12",
"@dev_zio_zio_managed_2_12",
"@org_apache_kafka_kafka_clients",
],
)
Loading