You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@apisix.apache.org by GitBox <gi...@apache.org> on 2021/09/23 16:27:26 UTC

[GitHub] [apisix-java-plugin-runner] tzssangglass opened a new pull request #73: fix: convert the conf req to an object and put it in the cache

tzssangglass opened a new pull request #73:
URL: https://github.com/apache/apisix-java-plugin-runner/pull/73


   Please answer these questions before submitting a pull request
   
   - Why submit this pull request?
   - [x] Bugfix
   - [ ] New feature provided
   - [ ] Improve performance
   
   - Related issues
   fix: https://github.com/apache/apisix-java-plugin-runner/issues/63
   ___
   ### Bugfix
   - Description
   Conf Req is currently put directly into the cache, but Conf Req is a DirectByteBuffer and multiple reads in different Http Call Reqs will cause exceptions
   - How to fix?
   Convert the conf req to an object and put it in the cache to avoid errors caused by reading the buffer multiple times.
   
   ___
   ### New feature or improvement
   - Describe the details and related test reports.
   
   - Source branch
   
   - Related commits and pull requests
   
   - Target branch
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-java-plugin-runner] tzssangglass merged pull request #73: fix: convert the conf req to an object and put it in the cache

Posted by GitBox <gi...@apache.org>.
tzssangglass merged pull request #73:
URL: https://github.com/apache/apisix-java-plugin-runner/pull/73


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-java-plugin-runner] tzssangglass commented on pull request #73: fix: convert the conf req to an object and put it in the cache

Posted by GitBox <gi...@apache.org>.
tzssangglass commented on pull request #73:
URL: https://github.com/apache/apisix-java-plugin-runner/pull/73#issuecomment-925978430


   relate to: https://github.com/apache/apisix-java-plugin-runner/issues/72


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-java-plugin-runner] huang-x-h commented on pull request #73: fix: convert the conf req to an object and put it in the cache

Posted by GitBox <gi...@apache.org>.
huang-x-h commented on pull request #73:
URL: https://github.com/apache/apisix-java-plugin-runner/pull/73#issuecomment-926479014


   There is no problem for the time being, continue to observe for a few days
   
   > @huang-x-h @Rdebu pls verify this PR if fix your problem.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-java-plugin-runner] tzssangglass commented on pull request #73: fix: convert the conf req to an object and put it in the cache

Posted by GitBox <gi...@apache.org>.
tzssangglass commented on pull request #73:
URL: https://github.com/apache/apisix-java-plugin-runner/pull/73#issuecomment-926484603


   @seominho77 @dongqisilent verify this PR if fix your problem.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [apisix-java-plugin-runner] tzssangglass commented on pull request #73: fix: convert the conf req to an object and put it in the cache

Posted by GitBox <gi...@apache.org>.
tzssangglass commented on pull request #73:
URL: https://github.com/apache/apisix-java-plugin-runner/pull/73#issuecomment-926361836


   @huang-x-h @Rdebu pls verify this PR if fix your problem.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@apisix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org