You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by GitBox <gi...@apache.org> on 2020/07/08 20:27:20 UTC

[GitHub] [geode] kohlmu-pivotal opened a new pull request #5357: GEODE-8067: ClassLoader Isolation

kohlmu-pivotal opened a new pull request #5357:
URL: https://github.com/apache/geode/pull/5357


   Thank you for submitting a contribution to Apache Geode.
   
   In order to streamline the review of the contribution we ask you
   to ensure the following steps have been taken:
   
   ### For all changes:
   - [ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?
   
   - [ ] Has your PR been rebased against the latest commit within the target branch (typically `develop`)?
   
   - [ ] Is your initial contribution a single, squashed commit?
   
   - [ ] Does `gradlew build` run cleanly?
   
   - [ ] Have you written or updated unit tests to verify your changes?
   
   - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)?
   
   ### Note:
   Please ensure that once the PR is submitted, check Concourse for build issues and
   submit an update to your PR as soon as possible. If you need help, please send an
   email to dev@geode.apache.org.
   


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-656405998


   This pull request **introduces 2 alerts** and **fixes 2** when merging fa991b29babee1c160fd8e734d56f990750d85fe into daa70d729b98f8edc3791a0dfccbf102ab94dd94 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-49691015b85b3016f0cd12c4b48d968180a69392)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-656876013


   This pull request **introduces 2 alerts** and **fixes 2** when merging 043ed5209b6da71f06200b844a8885aef38214fb into daa70d729b98f8edc3791a0dfccbf102ab94dd94 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-a17e5d99a4cc779049e05697bde7a3cbe6608e5e)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] kohlmu-pivotal closed pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
kohlmu-pivotal closed pull request #5357:
URL: https://github.com/apache/geode/pull/5357


   


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-657888811


   This pull request **introduces 2 alerts** and **fixes 2** when merging 591c7469807ff61ab71176003f906c986845957a into c41e3b4b559bfbc744c8c21844cd126de2ad2fb9 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-89c2bb8fdbbbb7b4c28a26432c538ccc3f17af02)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-655760929


   This pull request **introduces 2 alerts** and **fixes 2** when merging 55702129db394f0c5aab451aa9e99d59baafc797 into 9d7d467e642a3f84094d1bf6ef605079d8aa58aa - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-c44140c8f7558d58f3efa2d85a52283d333ddc84)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-657836650


   This pull request **fixes 2 alerts** when merging ff1286006fddc924485d7668155cc56c58a63933 into c41e3b4b559bfbc744c8c21844cd126de2ad2fb9 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-ffc286a7b66fc13ae218b8aa90d7a9e74fd0b606)
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-655793984


   This pull request **introduces 2 alerts** and **fixes 2** when merging 396f8cc794b0aa0b50eba0368499d59fab74de8e into 9d7d467e642a3f84094d1bf6ef605079d8aa58aa - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-af3315a6d29d9c13883c84501d4684b8070720d0)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-657812961


   This pull request **introduces 2 alerts** and **fixes 2** when merging 9cb762ed91581557c8c0fca0ac8983834a8e595c into c41e3b4b559bfbc744c8c21844cd126de2ad2fb9 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-4bdd1467f1ba8de5c3639c0500e1641205fbfb8f)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-656430092


   This pull request **introduces 2 alerts** and **fixes 2** when merging 05510648ef7e1710fa2fd9cf5c0b51b7728b4bb2 into daa70d729b98f8edc3791a0dfccbf102ab94dd94 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-6fae7a00aab914b200c0c60eb166f81647a31270)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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



[GitHub] [geode] lgtm-com[bot] commented on pull request #5357: GEODE-8067: ClassLoader Isolation

Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5357:
URL: https://github.com/apache/geode/pull/5357#issuecomment-655823212


   This pull request **introduces 2 alerts** and **fixes 2** when merging 175ac1b887df756aae20d94e53a80ff9101fc8ce into 9d7d467e642a3f84094d1bf6ef605079d8aa58aa - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-40250bcbb7e2f4da9c7849b278a7f027ed2023cb)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   
   **fixed alerts:**
   
   * 2 for Unused variable, import, function or class


----------------------------------------------------------------
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.

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