[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15588131#comment-15588131 ] Michael Dürig commented on OAK-4617: Filed OAK-4957 for this. > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587930#comment-15587930 ] Alex Parvulescu commented on OAK-4617: -- also found a TODO on {{SegmentRevisionGCMBean}} about this issue, we should drop that as well when we close this. > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587926#comment-15587926 ] Alex Parvulescu commented on OAK-4617: -- OAK-4956 > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587910#comment-15587910 ] Alex Parvulescu commented on OAK-4617: -- :) > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587909#comment-15587909 ] Michael Dürig commented on OAK-4617: Good catch! This is an oversight and we should fix it. Could you report an issue for this? > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587902#comment-15587902 ] Michael Dürig commented on OAK-4617: No please don't ;-) OAK-4941 is just the leftover technical task from OAK-4919 pending on the next Oak release. I would suggest to keep discussing improvements here or on a dedicated issue in itself. > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587895#comment-15587895 ] Alex Parvulescu commented on OAK-4617: -- Just checked again. The {{SegmentGCOptions#retainedGenerations}} is not exposed as OSGi, it can be updated via JMX though. > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15587893#comment-15587893 ] Alex Parvulescu commented on OAK-4617: -- [~volteanu] please add this to the dedicated issue OAK-4941 > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15584895#comment-15584895 ] Valentin Olteanu commented on OAK-4617: --- Regarding this, the current "Status" is showing the last log info. This is useful, but it would also be interesting to expose the real-time status. For monitoring it would be useful to know exactly in which phase we are, e.g. a field showing on of the following: - idle - estimation - compaction - compaction-retry-1 - compaction-retry-2 - compaction-forcecompact - cleanup Do you think this is possible/useful to add it to this MBean? > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15584870#comment-15584870 ] Michael Dürig commented on OAK-4617: I think this is pretty complete. I just wanted to double check whether we expose all parameters relevant to gc from this MBean (cross check with {{{SegmentGCOptions}}, {{SegmentNodeStoreService}}. Also do we have enough endpoints for monitoring an ongoing garbage collection or do we still need to add something? Finally are the current method names clear enough and consistent across the board? > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15584836#comment-15584836 ] Alex Parvulescu commented on OAK-4617: -- [~mduerig] you have better visibility here, what is still missing? > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15562550#comment-15562550 ] Michael Dürig commented on OAK-4617: Moved the functionality that was so far provided by the {{GCMonitorMBean}} to the {{SegmentRevisionGCMBean}} and removed the former at http://svn.apache.org/viewvc?rev=1764117&view=rev > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15561888#comment-15561888 ] Michael Dürig commented on OAK-4617: See http://markmail.org/message/pjb6psc46p2thd76 for my proposal for an initial step. > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig >Assignee: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (OAK-4617) Align SegmentRevisionGC MBean with new generation based GC
[ https://issues.apache.org/jira/browse/OAK-4617?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15551255#comment-15551255 ] Michael Dürig commented on OAK-4617: We also need to consider the outcome of OAK-4899 here. I.e. whether {{SegmentRevisionGCMBean.stopCompaction}} should be moved to those other MBeans, duplicated there or neither of both. > Align SegmentRevisionGC MBean with new generation based GC > -- > > Key: OAK-4617 > URL: https://issues.apache.org/jira/browse/OAK-4617 > Project: Jackrabbit Oak > Issue Type: Task > Components: segment-tar >Reporter: Michael Dürig > Labels: production > Fix For: Segment Tar 0.0.16 > > > The {{SegmentRevisionGC}} MBean still dates back to the old {{oak-segment}}. > We need to review its endpoints and only keep those that make sense for > {{oak-segment-tar}}, adapt the others as necessary any add further > functionality as required. -- This message was sent by Atlassian JIRA (v6.3.4#6332)