phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "James Taylor (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PHOENIX-1452) Add Phoenix client-side logging and capture resource utilization metrics
Date Fri, 06 Mar 2015 00:05:38 GMT

    [ https://issues.apache.org/jira/browse/PHOENIX-1452?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14349656#comment-14349656
] 

James Taylor commented on PHOENIX-1452:
---------------------------------------

LGTM. +1 with one extremely minor nit, add an logger.isDebugEnabled() within this if block
specific to the log line, like this:
{code}
+        if (PhoenixMetrics.isMetricsEnabled() || logger.isDebugEnabled()) {
+            for (Mutation mutation : mutations) {
+                byteSize += mutation.heapSize();
             }
+            MUTATION_BYTES.update(byteSize);
+            if (logger.isDebugEnabled()) logger.debug(LogUtil.addCustomAnnotations("Sending
" + mutations.size() + " mutations for " + Bytes.toString(htable.getTableName()) + " with
" + keyValueCount + " key values of total size " + byteSize + " bytes", connection));
         }
{code}

As far as going into 4.3, this commit would not break any b/w compatibility in terms of being
release on client/server in any order, so it's certainly possible. Would you be up for volunteering
to be the RM for the 4.3 release so that you can drive the consensus building for decisions
like that?


> Add Phoenix client-side logging and capture resource utilization metrics
> ------------------------------------------------------------------------
>
>                 Key: PHOENIX-1452
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-1452
>             Project: Phoenix
>          Issue Type: New Feature
>    Affects Versions: 5.0.0, 4.2
>            Reporter: Jan Fernando
>            Assignee: Samarth Jain
>         Attachments: PHOENIX-1452.patch, PHOENIX-1452_v2.patch, PHOENIX-1452_v3.patch,
PHOENIX-1452_v4.patch, wip.patch
>
>
> For performance testing and tuning of features that use Phoenix and for production monitoring
it would be really helpful to easily be able to extract statistics about Phoenix's client-side
Thread Pool and Queue Depth usage to help with tuning and being able to correlate the impact
of tuning these 2 parameters to query performance.
> For global per JVM logging one of the following would meet my needs, with a preference
for #2:
> 1. A simple log line that that logs the data in ThreadPoolExecutor.toString() at a configurable
interval
> 2. Exposing the ThreadPoolExecutor metrics in PhoenixRuntime or other global client exposed
class and allow client to do their own logging.
> In addition to this it would also be really valuable to have a single log line per query
that provides statistics about the level of parallelism i.e. number of parallel scans being
executed. I don't full explain plan level of data but a good heuristic to be able to track
over time how queries are utilizing the thread pool as data size grows etc. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message