[jboss-jira] [JBoss JIRA] (JGRP-2434) Support multiple TShark fields parser

Diego Lovison (Jira) issues at jboss.org
Tue Jan 14 16:40:06 EST 2020


     [ https://issues.redhat.com/browse/JGRP-2434?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Diego Lovison updated JGRP-2434:
--------------------------------
    Description: 
In other to achieve what I was looking for that is: have the time in front of the line.. it required too many changes in {{Util::parse}}

Changes
* Need to be divide the {{short version=dis.readShort();}} into the {{dis.read()}} to identify the {{\t}}
* {{Util::parse}} is used in GossipRouter

Why have a diff parser
* Prevent changing production code
* Easy to create/add more fields related only with TShark
* It reuse the {{Util::parse}}

  was:
In other to achieve what I was looking for that is: have the time in front of the line.. it required too many changes in {{Util::parse}}

Changes
* Need to be divide the {{short version=dis.readShort();}} into the {{dis.read()}} to identify the {{\t}}
* {{Util::parse}} is used in GossipRouter

Why have a diff parser
Prevent changing production code
Easy to create/add more fields related only with TShark
It reuse the {{Util::parse}}



> Support multiple TShark fields parser
> -------------------------------------
>
>                 Key: JGRP-2434
>                 URL: https://issues.redhat.com/browse/JGRP-2434
>             Project: JGroups
>          Issue Type: Enhancement
>    Affects Versions: 5.0
>            Reporter: Diego Lovison
>            Assignee: Diego Lovison
>            Priority: Major
>
> In other to achieve what I was looking for that is: have the time in front of the line.. it required too many changes in {{Util::parse}}
> Changes
> * Need to be divide the {{short version=dis.readShort();}} into the {{dis.read()}} to identify the {{\t}}
> * {{Util::parse}} is used in GossipRouter
> Why have a diff parser
> * Prevent changing production code
> * Easy to create/add more fields related only with TShark
> * It reuse the {{Util::parse}}



--
This message was sent by Atlassian Jira
(v7.13.8#713008)


More information about the jboss-jira mailing list