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

Separate thread for confluent kafka consumer #2003

Open
wants to merge 9 commits into
base: 0.6.0
Choose a base branch
from

Conversation

DABND19
Copy link

@DABND19 DABND19 commented Dec 22, 2024

Description

Fixes #1904

Type of change

Please delete options that are not relevant.

  • Documentation (typos, code examples, or any documentation updates)
  • Bug fix (a non-breaking change that resolves an issue)
  • New feature (a non-breaking change that adds functionality)
  • Breaking change (a fix or feature that would disrupt existing functionality)
  • This change requires a documentation update

Checklist

  • My code adheres to the style guidelines of this project (scripts/lint.sh shows no errors)
  • I have conducted a self-review of my own code
  • I have made the necessary changes to the documentation
  • My changes do not generate any new warnings
  • I have added tests to validate the effectiveness of my fix or the functionality of my new feature
  • Both new and existing unit tests pass successfully on my local environment by running scripts/test-cov.sh
  • I have ensured that static analysis tests are passing by running scripts/static-analysis.sh
  • I have included code examples to illustrate the modifications

dependabot bot and others added 9 commits December 9, 2024 12:14
…rtai#1978)

Updates the requirements on [typer](https://github.com/fastapi/typer), [mkdocs-material](https://github.com/squidfunk/mkdocs-material), [ruff](https://github.com/astral-sh/ruff), [bandit](https://github.com/PyCQA/bandit), [semgrep](https://github.com/returntocorp/semgrep), [coverage[toml]](https://github.com/nedbat/coveragepy), [pytest](https://github.com/pytest-dev/pytest), [fastapi](https://github.com/fastapi/fastapi) and [httpx](https://github.com/encode/httpx) to permit the latest version.

Updates `typer` to 0.15.1
- [Release notes](https://github.com/fastapi/typer/releases)
- [Changelog](https://github.com/fastapi/typer/blob/master/docs/release-notes.md)
- [Commits](fastapi/typer@0.9.0...0.15.1)

Updates `mkdocs-material` from 9.5.46 to 9.5.48
- [Release notes](https://github.com/squidfunk/mkdocs-material/releases)
- [Changelog](https://github.com/squidfunk/mkdocs-material/blob/master/CHANGELOG)
- [Commits](squidfunk/mkdocs-material@9.5.46...9.5.48)

Updates `ruff` from 0.8.0 to 0.8.2
- [Release notes](https://github.com/astral-sh/ruff/releases)
- [Changelog](https://github.com/astral-sh/ruff/blob/main/CHANGELOG.md)
- [Commits](astral-sh/ruff@0.8.0...0.8.2)

Updates `bandit` from 1.7.10 to 1.8.0
- [Release notes](https://github.com/PyCQA/bandit/releases)
- [Commits](PyCQA/bandit@1.7.10...1.8.0)

Updates `semgrep` from 1.97.0 to 1.99.0
- [Release notes](https://github.com/returntocorp/semgrep/releases)
- [Changelog](https://github.com/semgrep/semgrep/blob/develop/CHANGELOG.md)
- [Commits](semgrep/semgrep@v1.97.0...v1.99.0)

Updates `coverage[toml]` from 7.6.1 to 7.6.9
- [Release notes](https://github.com/nedbat/coveragepy/releases)
- [Changelog](https://github.com/nedbat/coveragepy/blob/master/CHANGES.rst)
- [Commits](nedbat/coveragepy@7.6.1...7.6.9)

Updates `pytest` from 8.3.3 to 8.3.4
- [Release notes](https://github.com/pytest-dev/pytest/releases)
- [Changelog](https://github.com/pytest-dev/pytest/blob/main/CHANGELOG.rst)
- [Commits](pytest-dev/pytest@8.3.3...8.3.4)

Updates `fastapi` from 0.115.5 to 0.115.6
- [Release notes](https://github.com/fastapi/fastapi/releases)
- [Commits](fastapi/fastapi@0.115.5...0.115.6)

Updates `httpx` from 0.27.2 to 0.28.1
- [Release notes](https://github.com/encode/httpx/releases)
- [Changelog](https://github.com/encode/httpx/blob/master/CHANGELOG.md)
- [Commits](encode/httpx@0.27.2...0.28.1)

---
updated-dependencies:
- dependency-name: typer
  dependency-type: direct:production
  dependency-group: pip
- dependency-name: mkdocs-material
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
- dependency-name: ruff
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
- dependency-name: bandit
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: pip
- dependency-name: semgrep
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: pip
- dependency-name: coverage[toml]
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
- dependency-name: pytest
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
- dependency-name: fastapi
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: pip
- dependency-name: httpx
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: pip
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
https://aio-pika.readthedocs.io/ currently returns 404 Project not found and it appears the documentation is now hosted at https://docs.aio-pika.com/
)

* fix: when extra / present in virtual host name

e.g name of vh is  "/test"

there is no problenm if passing vh separately, result url will "amqp://root:123@localhost:5672//test"

but if pass vh in uri, result uri will 
"amqp://root:123@localhost:5672/test"
cause lstrip("/") calling and slash missing in result url, 
so replace with count 1 is ok

* add test for rabbit broker build_url

* refactor building vh and add tests for this

* Update test_url_builder.py

* Update test_url_builder.py

---------

Co-authored-by: Pastukhov Nikita <[email protected]>
* feat: add BatchBufferOverflowException

* docs: generate API References

* fix: use pre-commit

* Update producer.py

* chore: fix CI

* chore: fix CI

---------

Co-authored-by: spataphore1337 <[email protected]>
Co-authored-by: Pastukhov Nikita <[email protected]>
Co-authored-by: Nikita Pastukhov <[email protected]>
@DABND19 DABND19 marked this pull request as draft December 22, 2024 20:58
@DABND19 DABND19 marked this pull request as ready for review December 22, 2024 21:38
@Lancetnik Lancetnik changed the base branch from main to 0.6.0 December 23, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Feature: Confluent message consuming refactoring
6 participants