-
Notifications
You must be signed in to change notification settings - Fork 4.5k
Add troubleshooting in local development #2344
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
Add troubleshooting in local development #2344
Conversation
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.
Thanks for the PR! One question but LGTM otherwise.
|
||
For example: | ||
``` | ||
env JAVA_HOME=/usr/lib/jvm/java-15-openjdk ./gradlew :airbyte-integrations:connectors:your-connector-dir:build |
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 this point to Java 14?
|
||
For example: | ||
``` | ||
env JAVA_HOME=/usr/lib/jvm/java-15-openjdk ./gradlew :airbyte-integrations:connectors:your-connector-dir:build |
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.
env JAVA_HOME=/usr/lib/jvm/java-15-openjdk ./gradlew :airbyte-integrations:connectors:your-connector-dir:build | |
env JAVA_HOME=/usr/lib/jvm/java-14-openjdk ./gradlew :airbyte-integrations:connectors:your-connector-dir:build |
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.
Good call. I think I have java 15 on my machine and that's where I needed this command. It seems to work fine with 15.
What
Fix a couple of typos and add troubleshooting section in developing locally page to solve an issue I encountered.
How
The Java home folder needs to be explicitly defined.
This is a documentation update so I haven't gone through the checklist below.
Pre-merge Checklist
Recommended reading order
test.java
component.ts