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/09/24 01:47:56 UTC
[GitHub] [geode] kohlmu-pivotal opened a new pull request #5548: GEODE-8466: Introduction of ClassLoaderService as replacement
kohlmu-pivotal opened a new pull request #5548:
URL: https://github.com/apache/geode/pull/5548
for the direct feature usage of ClassLoader and ClassLoaderPath
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] onichols-pivotal commented on pull request #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
onichols-pivotal commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-773029132
This PR has been inactive for some while, can it be closed?
----------------------------------------------------------------
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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-713240394
This pull request **introduces 3 alerts** and **fixes 1** when merging 382bd52cc8354170542a96ca106f2038060e695f into 505eb3af1f61edd8dff9e199104c72987ef281ab - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-f49d8aeadbe17f53b3dbe7719a8c80d312a9685a)
**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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-714044302
This pull request **introduces 3 alerts** and **fixes 1** when merging ddb08eb0c1fb44075fde9908ed16dfa37bc8c349 into 4624211cf271daa1f3db07f7849a9186fbca7980 - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-a2fe0258544bf21eb317e1cfae796bd4dde8ec50)
**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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-725072980
This pull request **introduces 3 alerts** and **fixes 1** when merging a74cbadc040f65f3f54e2e1ec319ba0ed6ba1976 into 70b1ee8b98f1458620539c4a962e82f8ef35707b - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-3b54fbb2549d9a0835c60fa8a05c5a8b48111271)
**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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-712521167
This pull request **introduces 3 alerts** and **fixes 1** when merging d80228410dd76454e5bf61553efabd32df5bafbc into 505eb3af1f61edd8dff9e199104c72987ef281ab - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-9646bdfcbac381a818210ba2828bafada3ecfac1)
**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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-713256989
This pull request **introduces 3 alerts** and **fixes 1** when merging 312d7d40e1fa24833b809c91c54c609d7168a518 into 505eb3af1f61edd8dff9e199104c72987ef281ab - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-4563248e3d3bced7909843c109d342da2e65e1f3)
**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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-724913938
This pull request **introduces 3 alerts** and **fixes 1** when merging 846309e00049b107ca9e72c6b699e20eefdcdd66 into 70b1ee8b98f1458620539c4a962e82f8ef35707b - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-d1e671db8a7e7bf2d6907c3f4c88dc2c86aa0aa7)
**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] kohlmu-pivotal closed pull request #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
kohlmu-pivotal closed pull request #5548:
URL: https://github.com/apache/geode/pull/5548
----------------------------------------------------------------
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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-713300748
This pull request **introduces 3 alerts** and **fixes 1** when merging 3fa823a3a9385555da3bb5319c9ecc90dff0f558 into 505eb3af1f61edd8dff9e199104c72987ef281ab - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-3f3acc6842a80970464cdcf53fd56a2d0b76006e)
**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 #5548: GEODE-8466: Introduction of ClassLoaderService
Posted by GitBox <gi...@apache.org>.
lgtm-com[bot] commented on pull request #5548:
URL: https://github.com/apache/geode/pull/5548#issuecomment-713220581
This pull request **introduces 3 alerts** and **fixes 1** when merging 6f10c15912b1c193f4175ef7cc4b8c99986ecf31 into 505eb3af1f61edd8dff9e199104c72987ef281ab - [view on LGTM.com](https://lgtm.com/projects/g/apache/geode/rev/pr-327ce89ec2e329adc1402178e15dffabfdb0b4f2)
**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