accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-4178) include table ids in client writer threads
Date Mon, 04 Apr 2016 13:23:25 GMT

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

ASF GitHub Bot commented on ACCUMULO-4178:
------------------------------------------

Github user dlmarion commented on a diff in the pull request:

    https://github.com/apache/accumulo/pull/87#discussion_r58374030
  
    --- Diff: core/src/main/java/org/apache/accumulo/core/client/impl/TabletServerBatchWriter.java
---
    @@ -831,11 +814,16 @@ public void send(TabletServerMutations<Mutation> tsm) throws
AccumuloServerExcep
             try {
     
               long count = 0;
    -          for (List<Mutation> list : mutationBatch.values()) {
    -            count += list.size();
    +
    +          Set<Text> tableIds = new TreeSet<Text>();
    +          for (Map.Entry<KeyExtent,List<Mutation>> entry : mutationBatch.entrySet())
{
    +            count += entry.getValue().size();
    +            tableIds.add(entry.getKey().getTableId());
               }
    -          String msg = "sending " + String.format("%,d", count) + " mutations to " +
String.format("%,d", mutationBatch.size()) + " tablets at " + location;
    -          Thread.currentThread().setName(msg);
    +          StringBuilder msg = new StringBuilder();
    +          msg.append("sending ").append(String.format("%,d", count)).append(" mutations
to ").append(String.format("%,d", mutationBatch.size()))
    +              .append(" tablets for ").append(String.format("%,d", tableIds.size())).append("
tables ").append(tableIds).append(" to ").append(location);
    --- End diff --
    
    nitpick - suggest adding surrounding tableIds with [] or (). Or, remove tableIds.size()
from the output if you are going to list the tables.


> include table ids in client writer threads
> ------------------------------------------
>
>                 Key: ACCUMULO-4178
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-4178
>             Project: Accumulo
>          Issue Type: Improvement
>          Components: client
>    Affects Versions: 1.6.5
>            Reporter: Ivan Bella
>            Priority: Minor
>             Fix For: 1.6.6
>
>   Original Estimate: 3h
>  Remaining Estimate: 3h
>
> The client threads are renamed with a title denoting the number of mutations being sent,
the number of tablets being sent to and the tserver hostname.  It would be useful to include
the tables names or ids in this thread name when trying to diagnose issues related to hot
spotting within a system.



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

Mime
View raw message