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

Allow specifying state storage url for Source/Sink localrun #7930

Merged
merged 2 commits into from
Aug 30, 2020

Conversation

jerrypeng
Copy link
Contributor

No description provided.

@jerrypeng jerrypeng added this to the 2.7.0 milestone Aug 28, 2020
@jerrypeng jerrypeng self-assigned this Aug 28, 2020
@jerrypeng
Copy link
Contributor Author

/pulsarbot run-failure-checks

@jerrypeng jerrypeng force-pushed the add_source_sink_cli_state_url branch from 0000607 to 2dd1c54 Compare August 28, 2020 21:37
@@ -127,6 +127,8 @@ void processArguments() throws Exception {
@Parameters(commandDescription = "Run a Pulsar IO sink connector locally (rather than deploying it to the Pulsar cluster)")
protected class LocalSinkRunner extends CreateSink {

@Parameter(names = "--state-storage-service-url", description = "The URL for the state storage service (the default is Apache BookKeeper)")
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is this using the old naming scheme ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@aahmed-se this is the current naming scheme

@jerrypeng jerrypeng merged commit 1443c6b into apache:master Aug 30, 2020
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Sep 5, 2020
)

Co-authored-by: Jerry Peng <jerryp@splunk.com>
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Sep 5, 2020
)

Co-authored-by: Jerry Peng <jerryp@splunk.com>
lbenc135 pushed a commit to lbenc135/pulsar that referenced this pull request Sep 5, 2020
)

Co-authored-by: Jerry Peng <jerryp@splunk.com>
merlimat pushed a commit to merlimat/pulsar that referenced this pull request Dec 19, 2020
)

Co-authored-by: Jerry Peng <jerryp@splunk.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants