X-MimeOLE: Produced By Microsoft Exchange V6.5
Received: by onstor-exch02.onstor.net 
	id <01C7B2FC.488E0288@onstor-exch02.onstor.net>; Tue, 19 Jun 2007 21:31:42 -0800
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----_=_NextPart_001_01C7B2FC.488E0288"
References: <BB375AF679D4A34E9CA8DFA650E2B04E04360E80@onstor-exch02.onstor.net>
Content-class: urn:content-classes:message
Subject: FW: Please review
Date: Tue, 19 Jun 2007 21:28:00 -0800
Message-ID: <BB375AF679D4A34E9CA8DFA650E2B04E04344A8C@onstor-exch02.onstor.net>
X-MS-Has-Attach: 
X-MS-TNEF-Correlator: 
Thread-Topic: Please review
Thread-Index: Acey0r+sqZvIjkfoQBOS21UjermTpAAKQRwN
From: "Tim Gardner" <tim.gardner@onstor.com>
To: "Andy Sharp" <andy.sharp@onstor.com>

This is a multi-part message in MIME format.

------_=_NextPart_001_01C7B2FC.488E0288
Content-Type: text/plain;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

Andy,
=20
Did you direct Max to do this work? I would guess not.
I believe that we have more important things to do than this.
You and I will need to work with Jay to keep Max focused on task that =
are in the schedule.
I will do the requested code review but now this distraction has become =
a time sink for me as well.
=20
Tim
=20

________________________________

From: Maxim Kozlovsky
Sent: Tue 6/19/2007 5:34 PM
To: Tim Gardner
Subject: Please review



Change 24299 by maximk@maximk-13 on 2007/06/19 17:27:14 *pending*

        Add commands to allow restricting gathering of the KPI data

           to a subset of clients.

Affected files ...

... //depot/cougar/nfx-tree/code/sm-fs/fs-context.c#5 edit

... //depot/cougar/nfx-tree/code/sm-fs/fs-err.c#3 edit

... //depot/cougar/nfx-tree/code/sm-nfs/nfs-conn.c#4 edit

... //depot/cougar/nfx-tree/code/sm-pkt/acpu-conn-api.h#3 edit

... //depot/cougar/nfx-tree/code/sm-req-queue/req-queue-api.h#4 edit

... //depot/cougar/nfx-tree/code/sm-stats/kpi-api.c#5 edit

... //depot/cougar/nfx-tree/code/sm-stats/kpi-api.h#4 edit

... //depot/cougar/nfx-tree/code/sm-stats/kpi.h#4 edit

... //depot/cougar/nfx-tree/code/sm-stats/stats.c#4 edit

... //depot/cougar/nfx-tree/code/sm-tpl/tpl-rcv.c#4 edit

... //depot/cougar/nfx-tree/code/sm-ui/cmdtable.c#7 edit

... //depot/cougar/nfx-tree/code/sm-ui/uihdr.h#5 edit

... //depot/cougar/nfx-tree/code/ssc-nfxsh/Makefile#7 edit

... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times.txt#3 =
edit

... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track.t=
xt#1 add

... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient.txt#1 add

... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient_add.txt#1 add

... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient_delete.txt#1 add

... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient_show.txt#1 add

... //depot/cougar/nfx-tree/code/ssc-nfxsh/cmd_help.c#6 edit

... //depot/cougar/nfx-tree/code/ssc-nfxsh/cmd_stats.c#5 edit

... //depot/cougar/nfx-tree/code/ssc-nfxsh/kpi-ui.c#4 edit


------_=_NextPart_001_01C7B2FC.488E0288
Content-Type: text/html;
	charset="iso-8859-1"
Content-Transfer-Encoding: quoted-printable

<HTML dir=3Dltr><HEAD><TITLE>Please review</TITLE>=0A=
<META http-equiv=3DContent-Type content=3D"text/html; charset=3Dunicode">=0A=
<META content=3D"MSHTML 6.00.2800.1479" name=3DGENERATOR></HEAD>=0A=
<BODY>=0A=
<DIV id=3DidOWAReplyText60090 dir=3Dltr>=0A=
<DIV dir=3Dltr><FONT face=3DArial color=3D#000000 =
size=3D2>Andy,</FONT></DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2></FONT>&nbsp;</DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2>Did you direct Max to do this =
work? I would guess not.</FONT></DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2>I believe that we have more =
important things to do than this.</FONT></DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2>You and I will need to work =
with Jay to keep Max focused on task that are in the =
schedule.</FONT></DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2>I will do the requested code =
review but now this distraction has become a time sink for me as =
well.</FONT></DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2></FONT>&nbsp;</DIV>=0A=
<DIV dir=3Dltr><FONT face=3DArial size=3D2>Tim</FONT></DIV>=0A=
<DIV dir=3Dltr>&nbsp;</DIV></DIV>=0A=
<DIV dir=3Dltr><BR>=0A=
<HR tabIndex=3D-1>=0A=
<FONT face=3DTahoma size=3D2><B>From:</B> Maxim =
Kozlovsky<BR><B>Sent:</B> Tue 6/19/2007 5:34 PM<BR><B>To:</B> Tim =
Gardner<BR><B>Subject:</B> Please review<BR></FONT><BR></DIV>=0A=
<DIV>=0A=
<P dir=3Dltr><SPAN lang=3Den-us></SPAN><SPAN lang=3Den-us><FONT =
face=3DArial size=3D2>Change 24299 by maximk@maximk-13 on 2007/06/19 =
17:27:14 *pending*</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial =
size=3D2>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; Add commands to =
allow restricting gathering of the KPI data</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial =
size=3D2>&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp;&nbsp; to =
a subset of clients.</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>Affected =
files ...</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-fs/fs-context.c#5 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-fs/fs-err.c#3 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-nfs/nfs-conn.c#4 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-pkt/acpu-conn-api.h#3 =
edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-req-queue/req-queue-api.h#4 =
edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-stats/kpi-api.c#5 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-stats/kpi-api.h#4 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-stats/kpi.h#4 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-stats/stats.c#4 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-tpl/tpl-rcv.c#4 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-ui/cmdtable.c#7 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/sm-ui/uihdr.h#5 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/Makefile#7 edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times.txt#3 =
edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track.t=
xt#1 add</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient.txt#1 add</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient_add.txt#1 add</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient_delete.txt#1 add</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/agile-man/help_stats_times_track_c=
lient_show.txt#1 add</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/cmd_help.c#6 =
edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/cmd_stats.c#5 =
edit</FONT></SPAN></P>=0A=
<P dir=3Dltr><SPAN lang=3Den-us><FONT face=3DArial size=3D2>... =
//depot/cougar/nfx-tree/code/ssc-nfxsh/kpi-ui.c#4 =
edit</FONT></SPAN><SPAN lang=3Den-us></SPAN><SPAN =
lang=3Den-us></SPAN></P></DIV></BODY></HTML>
------_=_NextPart_001_01C7B2FC.488E0288--
