Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@OrezzerO
Copy link
Contributor

@OrezzerO OrezzerO commented Nov 7, 2022

Please answer these questions before submitting a pull request

  • Why submit this pull request?
  • Bugfix
  • New feature provided
  • Improve performance
  • Improve document
  • The quick start document is out of date. For example : the PluginFilter's signature and jre version.
  • Add a guide to remote debug with Java plugin in docker.

@tzssangglass tzssangglass changed the title Improve quick start document docs: improve quick start document Nov 8, 2022
FROM apache/apisix:${version}-debian

ADD apache-apisix-java-plugin-runner-0.1.0-bin.tar.gz /usr/local/
RUN apt -y install openjdk-11-jdk
Copy link
Member

Choose a reason for hiding this comment

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

could we use the base image with jdk 11?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I use the image of apisix , and jdk should in the same image. If i use the base image with jdk , I have to install apisix .

@tzssangglass tzssangglass merged commit 902defc into apache:main Nov 9, 2022
@tzssangglass
Copy link
Member

@OrezzerO merged, many thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants