[forces] comments on draft-ietf-forces-packet-parallelization-00

Jamal Hadi Salim <hadi@mojatatu.com> Tue, 15 July 2014 10:37 UTC

Return-Path: <hadi@mojatatu.com>
X-Original-To: forces@ietfa.amsl.com
Delivered-To: forces@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 3F9B11B2837 for <forces@ietfa.amsl.com>; Tue, 15 Jul 2014 03:37:20 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -0.079
X-Spam-Level:
X-Spam-Status: No, score=-0.079 tagged_above=-999 required=5 tests=[BAYES_40=-0.001, FM_FORGED_GMAIL=0.622, RCVD_IN_DNSWL_LOW=-0.7] autolearn=ham
Received: from mail.ietf.org ([4.31.198.44]) by localhost (ietfa.amsl.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id qqGgKBAQZC2c for <forces@ietfa.amsl.com>; Tue, 15 Jul 2014 03:37:16 -0700 (PDT)
Received: from mail-vc0-f177.google.com (mail-vc0-f177.google.com [209.85.220.177]) (using TLSv1 with cipher ECDHE-RSA-RC4-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 077D91B27EA for <forces@ietf.org>; Tue, 15 Jul 2014 03:37:15 -0700 (PDT)
Received: by mail-vc0-f177.google.com with SMTP id hy4so3000701vcb.22 for <forces@ietf.org>; Tue, 15 Jul 2014 03:37:15 -0700 (PDT)
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:from:date:message-id:subject:to:cc :content-type; bh=OmDNPKcJel27vBPW3Pbs4zSO67xrNEWY/5RZDekRWxk=; b=D3vgeLn7CsDASY9Sr1ZGZiZ4OGQVhqTxvm/nbUekyaJyR7s9E9XjIFDamWVy9LG3Zi tz76SGh4CF1vCgtk2AZNjiNwFITNxz9oDIgAs/0dMvqNwpnQ5JGZQumaLibmA+wD/PRV 1dWduBRLtUPKooAyERRcQnDbzCvI6yq0Qb3FIN6/Wxx9M+Ac1y3Eriu4K4N/tWeolTXP Vuz5+pSpTXOhafEOLTcT5EABf2j9K8DovRZvU6vg1tzJFR25r43geb22wHrBD00wjedj M/Tm7lx1Spx2KfdH3EddubfCdX9r7enjEaecvICgR3rZ9odt3ibTtSbMmJyqfEQOUyAl 6JyQ==
X-Gm-Message-State: ALoCoQlNiw8egnr6Adx9nGUWjyiYbIqYXWJzE6EatSxGBEUFtdRx+ZyUmnoUhL2vvGQ8BCIAX2Hy
X-Received: by 10.52.121.112 with SMTP id lj16mr7092542vdb.29.1405420635062; Tue, 15 Jul 2014 03:37:15 -0700 (PDT)
MIME-Version: 1.0
Received: by 10.58.235.65 with HTTP; Tue, 15 Jul 2014 03:36:54 -0700 (PDT)
From: Jamal Hadi Salim <hadi@mojatatu.com>
Date: Tue, 15 Jul 2014 06:36:54 -0400
Message-ID: <CAAFAkD9ty-oipbjeDkTN6BFVVCOH8x25XKTGrkVXApf1AdTuKg@mail.gmail.com>
To: Evangelos Haleplidis <ehalep@gmail.com>, Joel Halpern <jmh@joelhalpern.com>
Content-Type: multipart/mixed; boundary="089e01184dc2b6c81004fe38fdf8"
Archived-At: http://mailarchive.ietf.org/arch/msg/forces/XU0cJy1qfA7-YdRLvyzYUB8EFBQ
Cc: "forces@ietf.org" <forces@ietf.org>
Subject: [forces] comments on draft-ietf-forces-packet-parallelization-00
X-BeenThere: forces@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: ForCES WG mailing list <forces.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/forces>, <mailto:forces-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/forces/>
List-Post: <mailto:forces@ietf.org>
List-Help: <mailto:forces-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/forces>, <mailto:forces-request@ietf.org?subject=subscribe>
X-List-Received-Date: Tue, 15 Jul 2014 10:37:20 -0000

Evangelos/Joel,

Since this doc is in WG LC i made a more thorough run of the text.
There are a few editorial comments that i would urge a version -01.
I have attached my comments against the draft.

Issues i would like to highlight:
1) You need to explicitly state there can be only one instance per FE.

2) You need to define the concept of a "task" which is equivalent to
your description of "work". A task is a grouping of packets or chunks initiated
at the splitter and converged at the merger. All packets/chunks of the same
task will have the same correlator. Or something along those lines.
Once that is defined, text readability improves when it is used later.

3) You need to specify/suggest what should happen if threshold event
counters are exceeded

4) The Merge LFB and MergeWaitType component:
Is there a timeout period that a merger is supposed to wait for a
task's packet/chunks
to arrive before declaring a fail?

cheer,
jamal