Re: SkyWalking Podling Report - January 2018

2018-01-03 Thread Willem Jiang
FYI, I just signed the report. Willem Jiang Blog: http://willemjiang.blogspot.com (English) http://jnn.iteye.com (Chinese) Twitter: willemjiang Weibo: 姜宁willem On Wed, Jan 3, 2018 at 8:16 PM, mck wrote: > > > This email was sent by an automated system on behalf

[GitHub] zone1511 commented on issue #721: Recommendation on storing/retrieving plugin specific configuration

2018-01-03 Thread GitBox
zone1511 commented on issue #721: Recommendation on storing/retrieving plugin specific configuration URL: https://github.com/apache/incubator-skywalking/issues/721#issuecomment-355039338 As related to another thread, I would like to build a conditional sql logging plugin which only log

[GitHub] zone1511 commented on issue #720: access AbstractTracerContext from ContextManager

2018-01-03 Thread GitBox
zone1511 commented on issue #720: access AbstractTracerContext from ContextManager URL: https://github.com/apache/incubator-skywalking/issues/720#issuecomment-355040095 yes, im interest to extend skywalking to add more analytic functions.

[GitHub] peng-yongsheng opened a new pull request #724: A big change in collector.

2018-01-03 Thread GitBox
peng-yongsheng opened a new pull request #724: A big change in collector. URL: https://github.com/apache/incubator-skywalking/pull/724 Changed the structures of the projects 1. Deleted the collector queue module, use Datacarrier only. Because of the Disruptor consume the message

[GitHub] wu-sheng commented on issue #721: Recommendation on storing/retrieving plugin specific configuration

2018-01-03 Thread GitBox
wu-sheng commented on issue #721: Recommendation on storing/retrieving plugin specific configuration URL: https://github.com/apache/incubator-skywalking/issues/721#issuecomment-355174368 All recent jdbc-related plugins trace sql without any parameter value. If you want to develop private

[GitHub] wu-sheng commented on issue #720: access AbstractTracerContext from ContextManager

2018-01-03 Thread GitBox
wu-sheng commented on issue #720: access AbstractTracerContext from ContextManager URL: https://github.com/apache/incubator-skywalking/issues/720#issuecomment-355174851 The official core don't find any real scenario, which need this access way. Your requirements look like a custom way,

[GitHub] wu-sheng commented on issue #720: access AbstractTracerContext from ContextManager

2018-01-03 Thread GitBox
wu-sheng commented on issue #720: access AbstractTracerContext from ContextManager URL: https://github.com/apache/incubator-skywalking/issues/720#issuecomment-355174987 As I mentioned here : https://github.com/apache/incubator-skywalking/issues/721#issuecomment-355174368 > The

[GitHub] coveralls commented on issue #723: Add warning log outputs, if too many spans created in one segment.

2018-01-03 Thread GitBox
coveralls commented on issue #723: Add warning log outputs, if too many spans created in one segment. URL: https://github.com/apache/incubator-skywalking/pull/723#issuecomment-355185879 [![Coverage

[GitHub] coveralls commented on issue #724: A big change in collector.

2018-01-03 Thread GitBox
coveralls commented on issue #724: A big change in collector. URL: https://github.com/apache/incubator-skywalking/pull/724#issuecomment-355197200 [![Coverage Status](https://coveralls.io/builds/14893430/badge)](https://coveralls.io/builds/14893430) Coverage decreased (-0.06%)

[GitHub] wu-sheng opened a new pull request #725: Add incoming attr for application node.

2018-01-03 Thread GitBox
wu-sheng opened a new pull request #725: Add incoming attr for application node. URL: https://github.com/apache/incubator-skywalking/pull/725 Incoming request node, means User or outside system access the cluster from this. Recommend the UI generate a User node for each incoming node

[GitHub] coveralls commented on issue #725: Add incoming attr for application node.

2018-01-03 Thread GitBox
coveralls commented on issue #725: Add incoming attr for application node. URL: https://github.com/apache/incubator-skywalking/pull/725#issuecomment-355209852 [![Coverage Status](https://coveralls.io/builds/14894182/badge)](https://coveralls.io/builds/14894182) Coverage