-
Notifications
You must be signed in to change notification settings - Fork 58
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
Updated Developer Guide #16
Updated Developer Guide #16
Conversation
Signed-off-by: Joshua Palis <jpalis@amazon.com>
…ether Signed-off-by: Joshua Palis <jpalis@amazon.com>
DEVELOPER_GUIDE.md
Outdated
|
||
Run main script using `gradlew run`. | ||
## Publish OpenSearch feature/extensions Branch to Maven local | ||
The work done to support the extensions framework is located on the `Feature/Extensions` branch of the OpenSearch project. It is necessary to publish the dependencies of this branch to your local maven repository prior to running the extension on a seperate process. In order to do this, first navigate to the directory that OpenSearch has been cloned to, checkout the correct branch, e.g. `git checkout feature/extensions`. Then run `./gradlew publishToMavenLocal`. It is necessary to publish dependencies to a local maven repository until this branch is merged to `main`, at which point all dependencies will be published to Maven central. |
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.
Can we mention these steps in bullet format for better readability?
Something like with more details for each point:
- Clone OpenSearch
git clone
https://github.com/[your username]/OpenSearch.git``. - cd OpenSearch
git checkout feature/extensions
- Publish dependencies to mavenLocal using
./gradlew publishToMavenLocal
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.
Feature/Extensions
-> feature/extensions
DEVELOPER_GUIDE.md
Outdated
- [Git Clone OpenSearch-SDK Repo](#git-clone-OpenSearch-SDK-repo) | ||
- [Git Clone OpenSearch Repo](#git-clone-opensearch-repo) | ||
- [Publish OpenSearch Feature/Extensions branch to Maven local](#publish-opensearch-feature/extensions-branch-to-maven-local) | ||
- [Run Extension](#run-extension) |
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 [Run SDK]
DEVELOPER_GUIDE.md
Outdated
## Publish OpenSearch feature/extensions Branch to Maven local | ||
The work done to support the extensions framework is located on the `Feature/Extensions` branch of the OpenSearch project. It is necessary to publish the dependencies of this branch to your local maven repository prior to running the extension on a seperate process. In order to do this, first navigate to the directory that OpenSearch has been cloned to, checkout the correct branch, e.g. `git checkout feature/extensions`. Then run `./gradlew publishToMavenLocal`. It is necessary to publish dependencies to a local maven repository until this branch is merged to `main`, at which point all dependencies will be published to Maven central. | ||
|
||
### Run Extension |
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.
Run Extension
-> Run SDK
DEVELOPER_GUIDE.md
Outdated
|
||
### Run Extension | ||
|
||
Navigate to the directory that OpenSearch-SDK has been cloned to and run main script using `gradlew run`. |
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.
./gradlew run
Signed-off-by: Joshua Palis <jpalis@amazon.com>
Signed-off-by: Joshua Palis <jpalis@amazon.com>
|
||
- First navigate to the directory that OpenSearch has been cloned to | ||
- Checkout the correct branch, e.g. `git checkout feature/extensions`. | ||
- Run `./gradlew publishToMavenLocal`. |
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 next step should be run ./gradlew check
to make sure the build is successful
Signed-off-by: Joshua Palis <jpalis@amazon.com>
DEVELOPER_GUIDE.md
Outdated
@@ -46,7 +64,7 @@ Run tests : | |||
|
|||
### Send Message using Telnet |
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.
Instead of telnet should we mention about running OpenSearch in another terminal as this was written before the connection was established?
…/process. Added example logs of TCP handshake and plugin name request responses Signed-off-by: Joshua Palis <jpalis@amazon.com>
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.
This looks good. Thanks @joshpalis!
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.