You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@artemis.apache.org by "GChuf (via GitHub)" <gi...@apache.org> on 2026/06/11 10:10:38 UTC
[PR] ARTEMIS-6010 - Add JMH test for SecurityManagerUtil.authorize [artemis]
GChuf opened a new pull request, #6512:
URL: https://github.com/apache/artemis/pull/6512
(no comment)
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6115 - Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "jbertram (via GitHub)" <gi...@apache.org>.
jbertram commented on PR #6512:
URL: https://github.com/apache/artemis/pull/6512#issuecomment-4691665510
The commit message still references ARTEMIS-6010. It should reference ARTEMIS-6115 instead.
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6115 - Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "GChuf (via GitHub)" <gi...@apache.org>.
GChuf commented on PR #6512:
URL: https://github.com/apache/artemis/pull/6512#issuecomment-4694555443
Fixed
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6010 - Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "GChuf (via GitHub)" <gi...@apache.org>.
GChuf commented on PR #6512:
URL: https://github.com/apache/artemis/pull/6512#issuecomment-4689005019
Added [ARTEMIS-6115](https://issues.apache.org/jira/browse/ARTEMIS-6115)
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6115 Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "jbertram (via GitHub)" <gi...@apache.org>.
jbertram merged PR #6512:
URL: https://github.com/apache/artemis/pull/6512
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6010 - Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "GChuf (via GitHub)" <gi...@apache.org>.
GChuf commented on PR #6512:
URL: https://github.com/apache/artemis/pull/6512#issuecomment-4679504251
Adding some results here:
testMixedRealisticLoad
```
main:
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 8 8 thrpt 4 1447724.998 ± 131903.760 ops/s
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 8 64 thrpt 4 1469255.196 ± 112481.214 ops/s
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 1024 8 thrpt 4 10725.773 ± 969.388 ops/s
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 1024 64 thrpt 4 3370.991 ± 675.485 ops/s
ARTEMIS-6010:
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 8 8 thrpt 4 1987273.479 ± 418166.511 ops/s (+37%)
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 8 64 thrpt 4 1982439.979 ± 132183.999 ops/s (+35%)
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 1024 8 thrpt 4 13689.542 ± 1068.121 ops/s (+27%)
SecurityManagerUtilAuthorizePerfTest.testMixedRealisticLoad 1024 64 thrpt 4 14257.800 ± 4640.701 ops/s (+323%)
testGuestSendDenied
main:
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 8 8 thrpt 4 1629657.175 ± 105127.627 ops/s
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 8 64 thrpt 4 1663939.770 ± 184689.908 ops/s
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 1024 8 thrpt 4 9070.053 ± 596.703 ops/s
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 1024 64 thrpt 4 2128.262 ± 260.978 ops/s
ARTEMIS-6010:
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 8 8 thrpt 4 2161555.224 ± 61623.114 ops/s (+33%)
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 8 64 thrpt 4 2064947.686 ± 172341.549 ops/s (+24%)
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 1024 8 thrpt 4 17034.768 ± 1383.279 ops/s (+87%)
SecurityManagerUtilAuthorizePerfTest.testGuestSendDenied 1024 64 thrpt 4 15933.721 ± 825.356 ops/s (+649%)
```
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6115 Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "jbertram (via GitHub)" <gi...@apache.org>.
jbertram commented on code in PR #6512:
URL: https://github.com/apache/artemis/pull/6512#discussion_r3430273453
##########
.project:
##########
@@ -14,4 +14,15 @@
<natures>
<nature>org.eclipse.m2e.core.maven2Nature</nature>
</natures>
+ <filteredResources>
+ <filter>
+ <id>1781261832964</id>
+ <name></name>
+ <type>30</type>
+ <matcher>
+ <id>org.eclipse.core.resources.regexFilterMatcher</id>
+ <arguments>node_modules|\.git|__CREATED_BY_JAVA_LANGUAGE_SERVER__</arguments>
+ </matcher>
+ </filter>
+ </filteredResources>
Review Comment:
This should be removed from this commit.
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6115 Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "GChuf (via GitHub)" <gi...@apache.org>.
GChuf commented on code in PR #6512:
URL: https://github.com/apache/artemis/pull/6512#discussion_r3434075606
##########
.project:
##########
@@ -14,4 +14,15 @@
<natures>
<nature>org.eclipse.m2e.core.maven2Nature</nature>
</natures>
+ <filteredResources>
+ <filter>
+ <id>1781261832964</id>
+ <name></name>
+ <type>30</type>
+ <matcher>
+ <id>org.eclipse.core.resources.regexFilterMatcher</id>
+ <arguments>node_modules|\.git|__CREATED_BY_JAVA_LANGUAGE_SERVER__</arguments>
+ </matcher>
+ </filter>
+ </filteredResources>
Review Comment:
Sorry, this one always tried to make it into my commits.
Fixed.
##########
.project:
##########
@@ -14,4 +14,15 @@
<natures>
<nature>org.eclipse.m2e.core.maven2Nature</nature>
</natures>
+ <filteredResources>
+ <filter>
+ <id>1781261832964</id>
+ <name></name>
+ <type>30</type>
+ <matcher>
+ <id>org.eclipse.core.resources.regexFilterMatcher</id>
+ <arguments>node_modules|\.git|__CREATED_BY_JAVA_LANGUAGE_SERVER__</arguments>
+ </matcher>
+ </filter>
+ </filteredResources>
Review Comment:
Sorry, this one always tries to make it into my commits.
Fixed.
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org
Re: [PR] ARTEMIS-6010 - Add JMH test for SecurityManagerUtil.authorize [artemis]
Posted by "jbertram (via GitHub)" <gi...@apache.org>.
jbertram commented on PR #6512:
URL: https://github.com/apache/artemis/pull/6512#issuecomment-4681824768
ARTEMIS-6010 is fixed/closed so you'll need to associate this PR with a new Jira.
--
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: issues-unsubscribe@artemis.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@artemis.apache.org
For additional commands, e-mail: issues-help@artemis.apache.org