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.

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 | Tue Jul 18 09:05:14 UTC 2023 #565

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
ce2732d
Ds scalafmt (#32485)
leonbur Feb 6, 2023
ac328aa
Remove unused test deps (#33323)
liucijus Mar 10, 2023
972e385
Blocking retries attempts tracking fix (#32757)
atamurius Mar 14, 2023
a9efd8b
Revert "Blocking retries attempts tracking fix" (#33818)
natansil Apr 3, 2023
d1eba67
[greyhound] remove internal topic creation - wix adapter (#33820)
berman7 Apr 3, 2023
648bf56
[greyhound] parallel consumer OffsetsAndGaps (#33605)
ben-wattelman Apr 4, 2023
2ad72b0
Blocking retries attempts tracking fix + fix transitive header bug (#…
natansil Apr 4, 2023
640a715
[greyhound] expose internal kafka producer metrics (#34160)
berman7 Apr 22, 2023
a3c110d
[greyhound] Cooperative Rebalance fix (#34153)
leonbur Apr 23, 2023
57e6fc4
code cleanup (#34171)
leonbur Apr 23, 2023
8548845
[greyhound] parallel consumer implementation (#34061)
ben-wattelman May 24, 2023
06949bc
[gh-consumers-proxy] s3 bridge (#34839)
berman7 May 29, 2023
574a7b2
[greyhound] parallel consumer - add visibility (#34908)
ben-wattelman May 30, 2023
6b2a978
[greyhound] parallel consumer visibility (#34926)
ben-wattelman Jun 1, 2023
213390d
[greyhound] parallel consumer - add offsets and gaps init (#35027)
ben-wattelman Jun 6, 2023
778c786
[greyhound] parallel consumer - add grouping for no-key records (#35071)
ben-wattelman Jun 11, 2023
c8af30c
make GreyhoundNG RetryConfig serializable (so can be sent over gRpc a…
natansil Jun 13, 2023
76f1a64
multi-tenant consumer proxy redesign - initial commit (#35244)
berman7 Jun 17, 2023
e616952
[greyhound] parallel consumer - add gaps limit (#35313)
ben-wattelman Jun 20, 2023
6a8f0a4
Fixes in preparation for ZIO 2.0.15 (#35320)
leonbur Jun 21, 2023
09f70da
[greyhound-consumer-proxy] start from latest offset when group doesn'…
berman7 Jun 22, 2023
ccc936e
[greyhound] parallel consumer - compression and encoding for gaps lim…
ben-wattelman Jun 25, 2023
9291971
greyhound proxy non blocking retries (#35456)
natansil Jun 29, 2023
590c085
new kafka monitor server for proxy (currently) (#35598)
berman7 Jul 2, 2023
918bae5
[greyhound] fix OffsetsInitializer metadata bug (#35684)
ben-wattelman Jul 6, 2023
ede8dd6
ConsumerIT: add a test: allow to override offsetReset with autoResetO…
natansil Jul 13, 2023
e4d177f
gh RecordConsumer - add visibility to extra properties setup with aut…
natansil Jul 13, 2023
98e23d5
gh RecordConsumer - allow to override offsetReset with autoResetOffse…
natansil Jul 16, 2023
30be463
[greyhound] gaps limit calculation script (#35898)
ben-wattelman Jul 16, 2023
aeb5452
[greyhound] parallel consumer - update gaps limit (#35902)
ben-wattelman Jul 16, 2023
6ccf80b
[gh-consumer-proxy] fix: if position is lower than beginning offset -…
berman7 Jul 16, 2023
5530a31
fix build for greyhound publish to maven (#35951)
natansil Jul 18, 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
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
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
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
949 changes: 558 additions & 391 deletions core/src/it/scala/com/wixpress/dst/greyhound/core/ConsumerIT.scala

Large diffs are not rendered by default.

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
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