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

chapter 2 scala examples #7

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

Conversation

sifraser
Copy link

The other half of the split containing the chapter two examples in Scala.

@williamsjj
Copy link
Collaborator

Having someone better at Scala review this. Should get it merged this week or early next.

val msgProperties = msgPropertiesBuilder.build

val channel = connection.createChannel
channel.confirmSelect
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think basic Producer from ch2 is not doing confirm selects. If your intention was to implement this https://github.com/rabbitinaction/sourcecode/blob/master/python/chapter-2/hello_world_producer_pubconfirm.py then you might want to rename the class/file

@sifraser
Copy link
Author

To be honest, I'd just worked through the book's example, I hadn't looked specifically at the Python examples. Have now added classes for the variations in the example.

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.

None yet

3 participants