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/10/23 00:34:20 UTC

[GitHub] [geode] yozaner1324 opened a new pull request #5658: Introduce classloaderservice

yozaner1324 opened a new pull request #5658:
URL: https://github.com/apache/geode/pull/5658


   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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 43000b9fa166477601cb64bb14dba9a7439e2c2d into af267c005a63317cbb8528cdb38eccf6a8747818 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-8eea1ae86ef56849af481e8b8e63b58fa5cd0422)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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] yozaner1324 closed pull request #5658: GEODE-8466: Introduce classloaderservice

Posted by GitBox <gi...@apache.org>.
yozaner1324 closed pull request #5658:
URL: https://github.com/apache/geode/pull/5658


   


----------------------------------------------------------------
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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 3bf7d2777f3aa5a1dec8e8c9bc4fb1f952264605 into e1f55187726c77a34d6204227be6336ef2aa9ec9 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-97d2141b3ae9cfd21a54fb354bf3e25e17bbd79b)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 84ec38df49295320007064c2fb3bbfe0596cd048 into 0fdfc1ad1e272eed9bf5e2ee59816b75b262d9c6 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-eb1b6911fc647ff55caefaa31ee4ef2cefa4f5ef)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 640cd3fb8c9a2b8ec25e6798393bf682ecced501 into c0689791876c2fda1e5fb4b1337daae3cbddcbe8 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-e4f6c00199d662fda0999ed23a0d0d12b794d492)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 846309e00049b107ca9e72c6b699e20eefdcdd66 into e1f55187726c77a34d6204227be6336ef2aa9ec9 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-cdf920e7abe98679643cb8a3e849ba1c94346119)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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] yozaner1324 closed pull request #5658: GEODE-8466: Introduce classloaderservice

Posted by GitBox <gi...@apache.org>.
yozaner1324 closed pull request #5658:
URL: https://github.com/apache/geode/pull/5658


   


----------------------------------------------------------------
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 #5658: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 9d12f2eb9bfee876a3241a12bf658a72f6f628ba into a048a7204f6110eedc97e20562f09463fbc9ce51 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-bb8ccbf4bd648b22f4df3e292b9e0b0e0c8535ef)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 30ece9bc340c2893797887fe6421c0c28ed654b6 into 39db7929cff991c8c07763a4a9b8391a1045a234 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-e584682641fe7c6af9f2b617ea5b2fc22b3308b2)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 19b1313d9d31dc3320e5659555649133e991db13 into 7367d17e3817fc41666d471c5eb4d0df0d33c18b - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-1bf6e8475bff02a5fe107ea9ef2ef16a37961649)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: GEODE-8466: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging 2f94c29b87ab4108b5ca55d7b6d49f384f7c6393 into 39db7929cff991c8c07763a4a9b8391a1045a234 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-c173806780d0ec648b95a0a6ca8f6525f5eb7a57)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


----------------------------------------------------------------
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 #5658: Introduce classloaderservice

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


   This pull request **introduces 3 alerts** and **fixes 1** when merging dfc5bbcbd1e409ff284bf04775f627303f9f186e into 0fdfc1ad1e272eed9bf5e2ee59816b75b262d9c6 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-00098becaed61a2d7c818752cfb1fc7bbf5896af)
   
   **new alerts:**
   
   * 2 for Potential input resource leak
   * 1 for Use of a broken or risky cryptographic algorithm
   
   **fixed alerts:**
   
   * 1 for Use of a broken or risky cryptographic algorithm


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