Re: Need contribution for October ASF Board report
No worries, I have added your contributions to the wiki and submitted the
reports.
Thank you again for taking the time to contribute.
Bruce
On Thu, Oct 11, 2018 at 11:46 PM Bruce Snyder
wrote:
> Hi Clebert,
>
> Thank you for the contributions. Can you please add this info directly to
> the wiki page?
>
> Bruce
>
> On Mon, Oct 1, 2018 at 9:19 AM Clebert Suconic
> wrote:
>
>> For ActiveMQ Artemis:
>>
>> - ActiveMQ Artemis 2.6.3 released
>> - Improvements in performance for AMQP in Artemis.
>> - Lots of bug fixes in 2.6.3 and 2.7.0
>> - Openwire has better performance now what will increase compatibility
>> between ActiveMQ 5 and Artemis.
>>
>> This is a soft bullet, Kind of subjective, I will let you decide on
>> bringing it or not:
>> - Feature gap between ActiveMQ 5 and Artemis has been addressed
>> release after release.
>> ( I don't have specifics now. Michael Pearce has committed one feature
>> here).
>> On Mon, Oct 1, 2018 at 2:05 AM Jean-Baptiste Onofré
>> wrote:
>> >
>> > Hi Bruce,
>> >
>> > Do you want me to add content directly on cwiki ?
>> >
>> > From my side, I have some works in progress on ActiveMQ:
>> >
>> > - Improved OSGi/Karaf support for Karaf 4.2.x
>> > - Improvements on the http transport connector (allowing more tuning on
>> > the Jetty server and connector)
>> > - Investigating some issues (on redelivery plugin for instance)
>> >
>> > Regards
>> > JB
>> >
>> > On 01/10/2018 02:10, Bruce Snyder wrote:
>> > > Please take a few minutes and contribute to this month's board report:
>> > >
>> > >
>> https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=93326087
>> > >
>> > > This report must be submitted to the board on Oct 10.
>> > >
>> > > Thank you in advance!
>> > >
>> > > Bruce
>> > >
>> >
>> > --
>> > Jean-Baptiste Onofré
>> > jbono...@apache.org
>> > http://blog.nanthrax.net
>> > Talend - http://www.talend.com
>>
>>
>>
>> --
>> Clebert Suconic
>>
>
>
> --
> perl -e 'print
> unpack("u30","D0G)U8V4\@4VYY9&5R\"F)R=6-E+G-N>61E
> ActiveMQ in Action: http://bit.ly/2je6cQ
> Blog: http://bsnyder.org/
Re: Website
Fair points, Martyn. I like the idea of a combination of JIRA issues plus a list of those issues that is easily visible somewhere such as the wiki. Seeing as how we are moving the website around completely, I believe we need to include the docs to support the older versions in some way. So, the first thought that came to mind is a search capability in order for users to find them easier. Another idea, we could also keep the old site intact and available as a sub-domain or URL, e.g., 5x.activemq.apache.org, activemq.apache.org/5.x, etc. However, the advantage of a search feature is that it would be useful across the site, not just for older stuff. We might even be able structure the search to allow users to select a version of the docs that they would like to search. Anyway, just some thoughts. I agree that we should try to get the new site functional ASAP, but that should not eliminate the old site entirely. We should not abandon users of older versions. Bruce On Fri, Oct 12, 2018 at 2:44 AM Martyn Taylor wrote: > Cheers gents, looks like we're all set with the git repos. > > Shall we start putting together a ToDo list for what needs to happen to > move to the new site? JIRA perhaps? > > Bruce you mentioned a search feature for older content. Can you elaborate > on this. Also, there's the actual content for the pages, key features, > users etc... Perhaps we can all contribute to these? I could take care of > the Artemis side of things, if people with more experience in NMS, CMS and > 5.x could come up with some content for the project home pages? > > I agree with Michael in that it'd be good to get the new site running even > if it's not 100% perfect then iteratively improve on it. > > Cheers > > On Thu, Oct 11, 2018 at 12:56 AM Clebert Suconic < > clebert.suco...@gmail.com> > wrote: > > > How to delete one now ? > > > > On Tue, Oct 9, 2018 at 6:55 PM Bruce Snyder > > wrote: > > > > > Whoops, I just created one as well: > > > > > > https://gitbox.apache.org/repos/asf?p=activemq-website.git > > > > > > Bruce > > > > > > On Tue, Oct 9, 2018 at 8:49 AM Clebert Suconic < > > clebert.suco...@gmail.com> > > > wrote: > > > > > > > New repo created: > > > > > > > > https://gitbox.apache.org/repos/asf?p=activemq-www.git > > > > On Tue, Oct 9, 2018 at 10:47 AM Clebert Suconic > > > > wrote: > > > > > > > > > > Robbie Gemmel pointed me to https://gitbox.apache.org/ > > > > > > > > > > I'm trying to create one now: > > > > > On Tue, Oct 9, 2018 at 10:36 AM Clebert Suconic > > > > > wrote: > > > > > > > > > > > > That's for Bruce Snyder then... > > > > > > On Tue, Oct 9, 2018 at 10:29 AM Justin Bertram < > > jbert...@redhat.com> > > > > wrote: > > > > > > > > > > > > > > There's a "Create a new Git repository" option on > > > > > > > https://selfserve.apache.org/. I clicked it and logged in, > but > > > > apparently > > > > > > > it's restricted to ASF members and PMC chairs. > > > > > > > > > > > > > > > > > > > > > Justin > > > > > > > > > > > > > > On Tue, Oct 9, 2018 at 9:24 AM Clebert Suconic < > > > > clebert.suco...@gmail.com> > > > > > > > wrote: > > > > > > > > > > > > > > > I create an Infra JIRA for this: > > > > > > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/INFRA-17124 > > > > > > > > On Tue, Oct 9, 2018 at 9:49 AM Clebert Suconic > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > I asked infra and no reponse. > > > > > > > > > > > > > > > > > > Does anyone know any procedure on creating a repo at > apache? > > > > > > > > > > > > > > > > > > > > > > > > > > > I did ask infra about something what to do.. but so far no > > > > response. I > > > > > > > > > know they are busy.. so if anyone here knows a better way > to > > > > handle > > > > > > > > > this > > > > > > > > > On Mon, Oct 8, 2018 at 9:14 AM Clebert Suconic > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > I just initiated a thread at apache infra list, asking > for > > > > guidance on > > > > > > > > > > how to setup things. > > > > > > > > > > > > > > > > > > > > will keep it posted here. > > > > > > > > > > On Mon, Oct 8, 2018 at 9:01 AM Christopher Shannon > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > Thanks for getting the website discussion restarted. I > > > > think it > > > > > > > > will be > > > > > > > > > > > quite beneficial if we can get the new website live and > > > have > > > > more > > > > > > > > clear > > > > > > > > > > > information on there such as what Artemis is, etc. > > > > > > > > > > > > > > > > > > > > > > On Sat, Oct 6, 2018 at 9:13 AM Clebert Suconic < > > > > > > > > clebert.suco...@gmail.com> > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > I will handle it next week. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 4, 2018 at 5:09 PM Bruce Snyder < > > > > > > > > bruce.sny...@gmail.com> > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > >
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224839730
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
--- End diff --
@clebertsuconic @wy96f
Not sure but deliverIfNecessary could be written simlar to this one?
```
private void deliverIfNecessary(Collection cursorList,
long minPage) {
for (PageSubscription cursor : cursorList) {
long firstPage = cursor.getFirstPage();
if (firstPage == minPage) {
if (cursor.getQueue().getMessageCount() == 0) {
cursor.getQueue().deliverAsync();
break;
}
}
}
}
```
As long as minPage is not Long.MAX_VALUE (it shouldn't be)...
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224832278
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
--- End diff --
this is saving a deliveryAsync call.. but I"m just wondering if it would be
too costly to do the check on every queue?
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224832160
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
--- End diff --
I was talking to @franz1981 here where I am.. couldn't we just call
deliverAsync on every queue after cleanup?
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224830396
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
+ slowSubscription.getQueue().deliverAsync();
--- End diff --
@wy96f @clebertsuconic totally agree!
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224828919
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
+ slowSubscription.getQueue().deliverAsync();
--- End diff --
@wy96f how do I get to meet you? :) IRC / talk... etc? you have been
providing great fixes.
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user clebertsuconic commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224827924
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
+ slowSubscription.getQueue().deliverAsync();
--- End diff --
@wy96f man, you're good!
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
Github user franz1981 commented on a diff in the pull request:
https://github.com/apache/activemq-artemis/pull/2369#discussion_r224825736
--- Diff:
artemis-server/src/main/java/org/apache/activemq/artemis/core/paging/cursor/impl/PageCursorProviderImpl.java
---
@@ -599,6 +600,29 @@ private long checkMinPage(Collection
cursorList) {
}
+ private void deliverIfNecessary(Collection
cursorList) {
+ long minPage = Long.MAX_VALUE;
+ PageSubscription slowSubscription = null;
+ int nonEmptyCursorNum = 0;
+
+ for (PageSubscription cursor : cursorList) {
+ long firstPage = cursor.getFirstPage();
+
+ // the cursor will return -1 if the cursor is empty
+ if (firstPage >= 0) {
+nonEmptyCursorNum++;
+if (firstPage < minPage) {
+ minPage = firstPage;
+ slowSubscription = cursor.getQueue().getMessageCount() == 0
? cursor : null;
+}
+ }
+ }
+
+ if (nonEmptyCursorNum > 1 && slowSubscription != null) {
+ slowSubscription.getQueue().deliverAsync();
--- End diff --
Just to understand the fix: given that QueueImpl::deliverAsync will trigger
QueueImpl::checkDepage that will schedule an async task QueueImpl::DepageRunner
how do you know that it will be finished time in order to have the
PageSubscription completed?
---
[GitHub] activemq-artemis pull request #2369: ARTEMIS-2123 Paging not stopped if ther...
GitHub user wy96f opened a pull request: https://github.com/apache/activemq-artemis/pull/2369 ARTEMIS-2123 Paging not stopped if there are no consumers on one subscription Reproduction steps: 1. create a topic t and two subscriptions ta, tb 2. send messages to both ta and tb 3. create ta consumers and receive all messages from ta 4. close consumers 5. only send message to tb 6. create tb consumers and receive all message from tb 7. topic not stopped paging You can merge this pull request into a Git repository by running: $ git pull https://github.com/wy96f/activemq-artemis not_stop_paging_without_consumers_on_subscription Alternatively you can review and apply these changes as the patch at: https://github.com/apache/activemq-artemis/pull/2369.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #2369 commit 6aa65c969073c2908520156ce5113a7c63a71151 Author: yang wei Date: 2018-10-12T12:32:02Z ARTEMIS-2123 Paging not stopped if there are no consumers on one subscription ---
Re: Website
Cheers gents, looks like we're all set with the git repos. Shall we start putting together a ToDo list for what needs to happen to move to the new site? JIRA perhaps? Bruce you mentioned a search feature for older content. Can you elaborate on this. Also, there's the actual content for the pages, key features, users etc... Perhaps we can all contribute to these? I could take care of the Artemis side of things, if people with more experience in NMS, CMS and 5.x could come up with some content for the project home pages? I agree with Michael in that it'd be good to get the new site running even if it's not 100% perfect then iteratively improve on it. Cheers On Thu, Oct 11, 2018 at 12:56 AM Clebert Suconic wrote: > How to delete one now ? > > On Tue, Oct 9, 2018 at 6:55 PM Bruce Snyder > wrote: > > > Whoops, I just created one as well: > > > > https://gitbox.apache.org/repos/asf?p=activemq-website.git > > > > Bruce > > > > On Tue, Oct 9, 2018 at 8:49 AM Clebert Suconic < > clebert.suco...@gmail.com> > > wrote: > > > > > New repo created: > > > > > > https://gitbox.apache.org/repos/asf?p=activemq-www.git > > > On Tue, Oct 9, 2018 at 10:47 AM Clebert Suconic > > > wrote: > > > > > > > > Robbie Gemmel pointed me to https://gitbox.apache.org/ > > > > > > > > I'm trying to create one now: > > > > On Tue, Oct 9, 2018 at 10:36 AM Clebert Suconic > > > > wrote: > > > > > > > > > > That's for Bruce Snyder then... > > > > > On Tue, Oct 9, 2018 at 10:29 AM Justin Bertram < > jbert...@redhat.com> > > > wrote: > > > > > > > > > > > > There's a "Create a new Git repository" option on > > > > > > https://selfserve.apache.org/. I clicked it and logged in, but > > > apparently > > > > > > it's restricted to ASF members and PMC chairs. > > > > > > > > > > > > > > > > > > Justin > > > > > > > > > > > > On Tue, Oct 9, 2018 at 9:24 AM Clebert Suconic < > > > clebert.suco...@gmail.com> > > > > > > wrote: > > > > > > > > > > > > > I create an Infra JIRA for this: > > > > > > > > > > > > > > > > > > > > > https://issues.apache.org/jira/browse/INFRA-17124 > > > > > > > On Tue, Oct 9, 2018 at 9:49 AM Clebert Suconic > > > > > > > wrote: > > > > > > > > > > > > > > > > I asked infra and no reponse. > > > > > > > > > > > > > > > > Does anyone know any procedure on creating a repo at apache? > > > > > > > > > > > > > > > > > > > > > > > > I did ask infra about something what to do.. but so far no > > > response. I > > > > > > > > know they are busy.. so if anyone here knows a better way to > > > handle > > > > > > > > this > > > > > > > > On Mon, Oct 8, 2018 at 9:14 AM Clebert Suconic > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > I just initiated a thread at apache infra list, asking for > > > guidance on > > > > > > > > > how to setup things. > > > > > > > > > > > > > > > > > > will keep it posted here. > > > > > > > > > On Mon, Oct 8, 2018 at 9:01 AM Christopher Shannon > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > Thanks for getting the website discussion restarted. I > > > think it > > > > > > > will be > > > > > > > > > > quite beneficial if we can get the new website live and > > have > > > more > > > > > > > clear > > > > > > > > > > information on there such as what Artemis is, etc. > > > > > > > > > > > > > > > > > > > > On Sat, Oct 6, 2018 at 9:13 AM Clebert Suconic < > > > > > > > clebert.suco...@gmail.com> > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > I will handle it next week. > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 4, 2018 at 5:09 PM Bruce Snyder < > > > > > > > bruce.sny...@gmail.com> > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > Hi Clebert, > > > > > > > > > > > > > > > > > > > > > > > > It would be very helpful if you could create a git > repo > > > for it > > > > > > > and get > > > > > > > > > > > the > > > > > > > > > > > > source code moved into there. > > > > > > > > > > > > > > > > > > > > > > > > Bruce > > > > > > > > > > > > > > > > > > > > > > > > On Thu, Oct 4, 2018 at 7:58 AM Clebert Suconic < > > > > > > > > > > > clebert.suco...@gmail.com> > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > @Bruce: Anything I can do to expedie this? or you > are > > > taking > > > > > > > care of > > > > > > > > > > > > > the private repo? > > > > > > > > > > > > > On Wed, Oct 3, 2018 at 1:52 AM michael.andre.pearce > > > > > > > > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > > > > > If you noted because its using jekyll i could use > > > github > > > > > > > pages. > > > > > > > > > > > > > > One option is you could host it on asf git, have > > the > > > github > > > > > > > clone > > > > > > > > > > > like > > > > > > > > > > > > > we do with code and then use github pages, simply > > > asking infra > > > > > > > to have > > > > > > > > > > > > the > > > > > > > > > > > > > sub domain point to the github pages. Not sure if > > tha
[GitHub] activemq-artemis issue #2181: ARTEMIS-1980 Warn on failed check of table exi...
Github user franz1981 commented on the issue: https://github.com/apache/activemq-artemis/pull/2181 @clebertsuconic If now it looks fine I would cherry-pick it into 2.6-x too, fater being merged :+1: ---
