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

bump to v1.9.1 #1

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

bump to v1.9.1 #1

wants to merge 30 commits into from

Conversation

denizsurmeli
Copy link

No description provided.

edenhill and others added 30 commits September 21, 2021 10:03
…olman, confluentinc#676)

* Added convenience method StoreMessage that calls StoreOffsets

* Added '.' at the end of the comment

* Added Offset less than 0 check

* Moved the check to the correct method

Co-authored-by: Finn Colman <[email protected]>
…confluentinc#729)

* Added MockCluster implementation which can be used for testing

* adjust documentation for mock cluster

* Added example for MockCluster

* Fixed naming

Co-authored-by: Magnus Edenhill <[email protected]>

* fixed typo

Co-authored-by: Magnus Edenhill <[email protected]>

* removed

Co-authored-by: Magnus Edenhill <[email protected]>

* fixed comment

Co-authored-by: Magnus Edenhill <[email protected]>

* adjust documentation

Co-authored-by: Magnus Edenhill <[email protected]>

* adjustments regarding comments from Magnus

* fixed cleanup code for kafka configuration

* fixed destroy for C references

* fixed spaces

* removed call to `C.rd_kafka_conf_destroy(mc.cConf)` in close

* added docu for MockCluster type

* moved package statement into first line

* adjustments according to @edenhill comments

* Update kafka/mockcluster.go

Co-authored-by: Magnus Edenhill <[email protected]>

* removed duplicate import

* removed the remark as suggested from @edelhill

* changed typedef from struct_rd_kafka_mock_cluster_s to rd_kafka_mock_cluster_t

Co-authored-by: Kristian Köhler <[email protected]>
Co-authored-by: Magnus Edenhill <[email protected]>
Co-authored-by: Kristian Köhler <[email protected]>
.. and remove Require: check from pc, will not work and is not needed.
The "package .." stanza needs to follow the package docs comment, but
a previous commit moved the stanza to the top of the file which caused the
package docs to be ignored by 'godoc'.
Bindings for ACL operations:
CreateAcls
DescribeAcls
DeleteAcls
changed references to the new go.dev site or to github.com
improvements to client examples:
- Changed the producer example to use the Events delivery channel.
- Added another producer example using a custom delivery channel
- Moved channel producer and consumer examples to the
  examples/legacy folder
- Added missing descriptions for other examples
- Consumer example with at-least-once guarantee by disabling
  enable.auto.offset.store and using StoreMessage
- Custom channel example with a channel for each Produce call
@denizsurmeli denizsurmeli changed the title bump to v1.9.0 bump to v1.9.1 Mar 26, 2024
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.

9 participants