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

Include pulsar-checksum in pulsar-client shaded jar #65

Merged
merged 1 commit into from
Oct 14, 2016

Conversation

merlimat
Copy link
Contributor

Motivation

Shaded jar for pulsar-client should mask all dependencies. We were missing the pulsar-checksum module that has been recently added as a dependency.

@yahoocla
Copy link

CLA is valid!

@merlimat merlimat added this to the 1.15 milestone Oct 14, 2016
Copy link
Contributor

@sboobna sboobna left a comment

Choose a reason for hiding this comment

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

👍

@merlimat merlimat merged commit bd49d69 into apache:master Oct 14, 2016
sijie added a commit to sijie/pulsar that referenced this pull request Mar 4, 2018
* Add worker main

* Organize function packages in hierarchy

- functions are stored under distributedlog://<zkservers>/pulsar/functions/<tenant>/<namespace>/<function>/<package_name>
- initialize dlog namespace at the startup of worker
- use zookeeperServers in worker config instead of dlogUri to hide dlog from pulsar configuration. make it consistent with broker configuration.

* Addressed comments
massakam pushed a commit to massakam/pulsar that referenced this pull request Aug 6, 2020
hangc0276 pushed a commit to hangc0276/pulsar that referenced this pull request May 26, 2021
---

Fixes apache#64

*Motivation*

Currently, kop protocol handler only prints header at debug
statements. We also need request and response.

*Modifications*

- Add more request and response info
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.

3 participants