-
Notifications
You must be signed in to change notification settings - Fork 83
Rename Neo4j connectors to network connectors to differentiate them from Kafka, Spark, etc. #1301
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
Conversation
|
Thanks for the documentation updates. The preview documentation has now been torn down - reopening this PR will republish it. |
|
|
||
| The connectors are configured by settings on the format `server.<connector-name>.<setting-suffix>>`. | ||
| The available suffixes are described in the table below: | ||
| The network connectors are configured by settings on the format `server.<network-connector-name>.<setting-suffix>>`. |
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 it should say: in the format
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.
yes, good spot
| It is possible to specify defaults for the configuration options with `listen_address` and `advertised_address` suffixes, as described below. | ||
| Setting a default value will apply to all the connectors, unless specifically configured for a certain connector. | ||
| It is possible to specify defaults for the configuration options with `listen_address` and `advertised_address` suffixes. | ||
| Setting a default value applies to all network connectors unless specifically configured for a certain connector. |
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.
Should be something like:
If a default value is set, it will apply to all
or
Default value are applied to all
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 haven't edited the text. I just fixed the future tense here, as this is how it works.
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've commented on a couple of places that maybe need adjusting.
Thanks for the quick response, @mauvo! I'll fix the first one and leave the second one as it is if you don't have a strong opinion about it. |
…rom Kafka, Spark, etc. (neo4j#1301)
Rename the section title according to the recent changes in page _Configure network connectors_. See PR #1301
Rename the section title according to the recent changes in page _Configure network connectors_. See PR neo4j#1301
No description provided.