-
Notifications
You must be signed in to change notification settings - Fork 171
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
[Doc] Use absolute path when installing functions or using localrun #877
Merged
Merged
Changes from all commits
Commits
Show all changes
5 commits
Select commit
Hold shift + click to select a range
a5741ce
Use absolute path when installing functions
lhotari a1e8ba3
Fix other relative paths
lhotari 8bd0e69
Address a few more locations
lhotari 85f615d
Fix even more relative paths and docs
lhotari 32b9dc3
Address review commend: remove mandatory sentence for classname
lhotari File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -31,7 +31,7 @@ The configuration of the Debezium source connector has the following properties. | |
| `database.history` | true | null | The name of the database history class. | | ||
| `database.history.pulsar.topic` | true | null | The name of the database history topic where the connector writes and recovers DDL statements. <br /><br />**Note: this topic is for internal use only and should not be used by consumers.** | | ||
| `database.history.pulsar.service.url` | false| null | Pulsar cluster service URL for history topic. <br /><br />**Note**: If `database.history.pulsar.service.url` is not set, then the database history Pulsar client will use the same client settings as those of the source connector, such as `client_auth_plugin` and `client_auth_params`.| | ||
| `pulsar.service.url` | true | null | Pulsar cluster service URL for the offset topic used in Debezium. You can use the `bin/pulsar-admin --admin-url http://pulsar:8080 sources localrun --source-config-file configs/pg-pulsar-config.yaml` command to point to the target Pulsar cluster.| | ||
| `pulsar.service.url` | true | null | Pulsar cluster service URL for the offset topic used in Debezium. You can use the `bin/pulsar-admin --admin-url http://pulsar:8080 sources localrun --source-config-file $PWD/configs/pg-pulsar-config.yaml` command to point to the target Pulsar cluster.| | ||
| `offset.storage.topic` | true | null | Record the last committed offsets that the connector successfully completes. | | ||
| `mongodb.hosts` | true | null | The comma-separated list of hostname and port pairs (in the form 'host' or 'host:port') of the MongoDB servers in the replica set. The list contains a single hostname and a port pair. If mongodb.members.auto.discover is set to false, the host and port pair are prefixed with the replica set name (e.g., rs0/localhost:27017). | | ||
| `mongodb.name` | true | null | A unique name that identifies the connector and/or MongoDB replica set or shared cluster that this connector monitors. Each server should be monitored by at most one Debezium connector, since this server name prefixes all persisted Kafka topics emanating from the MongoDB replica set or cluster. | | ||
|
@@ -138,7 +138,7 @@ This example shows how to change the data of a MySQL table using the Pulsar Debe | |
|
||
```bash | ||
bin/pulsar-admin source localrun \ | ||
--archive connectors/pulsar-io-debezium-mysql-@pulsar:[email protected] \ | ||
--archive $PWD/connectors/pulsar-io-debezium-mysql-@pulsar:[email protected] \ | ||
--name debezium-mysql-source \ | ||
--tenant public \ | ||
--namespace default \ | ||
|
@@ -149,7 +149,7 @@ This example shows how to change the data of a MySQL table using the Pulsar Debe | |
|
||
```bash | ||
bin/pulsar-admin source localrun \ | ||
--source-config-file debezium-mysql-source-config.yaml | ||
--source-config-file $PWD/debezium-mysql-source-config.yaml | ||
``` | ||
|
||
4. Subscribe to the topic _sub-products_ for the table _inventory.products_. | ||
|
@@ -289,7 +289,7 @@ This example shows how to change the data of a PostgreSQL table using the Pulsar | |
|
||
```bash | ||
bin/pulsar-admin source localrun \ | ||
--archive connectors/pulsar-io-debezium-postgres-@pulsar:[email protected] \ | ||
--archive $PWD/connectors/pulsar-io-debezium-postgres-@pulsar:[email protected] \ | ||
--name debezium-postgres-source \ | ||
--tenant public \ | ||
--namespace default \ | ||
|
@@ -300,7 +300,7 @@ This example shows how to change the data of a PostgreSQL table using the Pulsar | |
|
||
```bash | ||
bin/pulsar-admin source localrun \ | ||
--source-config-file debezium-postgres-source-config.yaml | ||
--source-config-file $PWD/debezium-postgres-source-config.yaml | ||
``` | ||
|
||
4. Subscribe to the topic _sub-products_ for the _inventory.products_ table. | ||
|
@@ -431,7 +431,7 @@ This example shows how to change the data of a MongoDB table using the Pulsar De | |
|
||
```bash | ||
bin/pulsar-admin source localrun \ | ||
--archive connectors/pulsar-io-debezium-mongodb-@pulsar:[email protected] \ | ||
--archive $PWD/connectors/pulsar-io-debezium-mongodb-@pulsar:[email protected] \ | ||
--name debezium-mongodb-source \ | ||
--tenant public \ | ||
--namespace default \ | ||
|
@@ -442,7 +442,7 @@ This example shows how to change the data of a MongoDB table using the Pulsar De | |
|
||
```bash | ||
bin/pulsar-admin source localrun \ | ||
--source-config-file debezium-mongodb-source-config.yaml | ||
--source-config-file $PWD/debezium-mongodb-source-config.yaml | ||
``` | ||
|
||
4. Subscribe to the topic _sub-products_ for the _inventory.products_ table. | ||
|
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is $PWD a safe assumption as to the proper location of the artifact, or should we use a more explicit nomenclature such as
path/to/examples/api-examples.jar
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think
$PWD
gives a good hint that it's an absolute path. At least I have found it useful withpulsar-admin
when I have the files in the current directory and I want to reference the files with an absolute path. I guess there are also other ways. Perhaps it could be explained to the user that$PWD
is the current working directory. They will also find out if they try it or google it. :)In this particular case, the original
/examples/api-examples.jar
doesn't seem to make much sense. That's why I added the$PWD
prefix.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The
/path/to/file.jar
type of examples have the problem that for the user, it's really a lot of extra steps to find out the absolute path for the file arguments. That's why I prefer$PWD/file.jar
since it's simple and if someone doesn't already know it, they have the chance to learn it.$PWD
is for a unix shell, but the examples are already assuming a unix shell since the syntax is all unix like.