[
https://issues.apache.org/jira/browse/HDDS-1154?focusedWorklogId=209462&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-209462
]
ASF GitHub Bot logged work on HDDS-1154:
----------------------------------------
Author: ASF GitHub Bot
Created on: 07/Mar/19 09:01
Start Date: 07/Mar/19 09:01
Worklog Time Spent: 10m
Work Description: elek commented on pull request #507: HDDS-1154. Add
tracing to the client side of StorageContainerLocationProtocol and
OzoneManagerProtocol
URL: https://github.com/apache/hadoop/pull/507
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 209462)
Time Spent: 0.5h (was: 20m)
> Add tracing to the client side of StorageContainerLocationProtocol and
> OzoneManagerProtocol
> -------------------------------------------------------------------------------------------
>
> Key: HDDS-1154
> URL: https://issues.apache.org/jira/browse/HDDS-1154
> Project: Hadoop Distributed Data Store
> Issue Type: Sub-task
> Reporter: Elek, Marton
> Assignee: Elek, Marton
> Priority: Major
> Labels: pull-request-available
> Fix For: 0.4.0
>
> Time Spent: 0.5h
> Remaining Estimate: 0h
>
> The best way to display the network performance in the tracing server is to
> start a new span (=save tracing information) on both of the client and and
> server side.
> This issue is about improving the client side tracing of OzoneManager and
> StorageLocationManager.
> The easiest way to turn on tracing for a class which implements an interface
> is to create a java proxy with TracingUtil.createProxy. With this utility we
> can execute the required tracing methods (create span/close span) around the
> original methods without adding any boilerplate code.
> Thanks to the current design hadoop rpc calls are create by the
> *ClientSideTranslatorPB classes which implements the original protocol
> interface (eg. StorageContainerLocationProtocolClientSideTranslatorPB
> implements StorageContainerLocationProtocol) which means that it can easily
> instrumented by TracingUtil.createProxy.
> The only thing what we need is to use the interface everywhere
> (StorageContainerLocationProtocol) instead of the implementation
> (ClientSideTranslator) as a client.
> The only required method which is not published in the ClientSideTranslator
> is the close method. With adding the close method to the interface (extends
> Closable) we are able to use the interface everywhere which can be
> instrumented to send the tracing information.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
