Re: [Gen-art] Gen-ART review of draft-ietf-ppsp-peer-protocol-10

Arno Bakker <abr800@vu.nl> Wed, 09 July 2014 14:45 UTC

Return-Path: <a.bakker@vu.nl>
X-Original-To: gen-art@ietfa.amsl.com
Delivered-To: gen-art@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 552971A040B for <gen-art@ietfa.amsl.com>; Wed, 9 Jul 2014 07:45:07 -0700 (PDT)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: -2.757
X-Spam-Level:
X-Spam-Status: No, score=-2.757 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, HELO_EQ_NL=0.55, HOST_EQ_NL=1.545, RCVD_IN_DNSWL_MED=-2.3, RP_MATCHES_RCVD=-0.651, SPF_PASS=-0.001] 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 lrtyqi9OOy0q for <gen-art@ietfa.amsl.com>; Wed, 9 Jul 2014 07:45:03 -0700 (PDT)
Received: from mailin.vu.nl (mailin.vu.nl [130.37.164.16]) (using TLSv1 with cipher AES128-SHA (128/128 bits)) (No client certificate requested) by ietfa.amsl.com (Postfix) with ESMTPS id 873781A0AEB for <gen-art@ietf.org>; Wed, 9 Jul 2014 07:45:01 -0700 (PDT)
Received: from PEXHB012A.vu.local (130.37.236.66) by mailin.vu.nl (130.37.164.16) with Microsoft SMTP Server (TLS) id 14.3.174.1; Wed, 9 Jul 2014 16:44:56 +0200
Received: from [10.0.0.103] (130.37.253.20) by mails.vu.nl (130.37.236.66) with Microsoft SMTP Server (TLS) id 14.3.174.1; Wed, 9 Jul 2014 16:44:59 +0200
Message-ID: <53BD556D.70200@vu.nl>
Date: Wed, 09 Jul 2014 16:45:01 +0200
From: Arno Bakker <abr800@vu.nl>
User-Agent: Mozilla/5.0 (Windows NT 6.3; WOW64; rv:24.0) Gecko/20100101 Thunderbird/24.6.0
MIME-Version: 1.0
To: Christer Holmberg <christer.holmberg@ericsson.com>, "gen-art@ietf.org" <gen-art@ietf.org>
References: <7594FB04B1934943A5C02806D1A2204B1D3AB180@ESESSMB209.ericsson.se>
In-Reply-To: <7594FB04B1934943A5C02806D1A2204B1D3AB180@ESESSMB209.ericsson.se>
Content-Type: text/plain; charset="ISO-8859-1"; format="flowed"
Content-Transfer-Encoding: 7bit
X-Originating-IP: [130.37.253.20]
Archived-At: http://mailarchive.ietf.org/arch/msg/gen-art/5nNpvT1Fj3IsYVmG11yx6qZR5Ew
X-Mailman-Approved-At: Wed, 09 Jul 2014 13:41:18 -0700
Cc: "draft-ietf-ppsp-peer-protocol.all@tools.ietf.org" <draft-ietf-ppsp-peer-protocol.all@tools.ietf.org>
Subject: Re: [Gen-art] Gen-ART review of draft-ietf-ppsp-peer-protocol-10
X-BeenThere: gen-art@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: "GEN-ART: General Area Review Team" <gen-art.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/gen-art>, <mailto:gen-art-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/gen-art/>
List-Post: <mailto:gen-art@ietf.org>
List-Help: <mailto:gen-art-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/gen-art>, <mailto:gen-art-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 09 Jul 2014 14:45:07 -0000

Hello Christer and Gen-ART

thanks for the positive review! Please see inline for our responses

Christer Holmberg schreef op 30-6-2014 12:07:
>
> Minor Issues:
>
> Section 3.12 talks about keep alive signaling.
>
> Q1: The sending of keep alives is a SHOULD, and there are no
> procedures on how to act if keep alives are not received. There isn't
> even a mechanism to negotiate the sending of keep alives.
>

If no keep alives are sent, a peer runs a risk of being declared dead 
following the conditions of Section 8.15. Note that if it does send 
normal messages, this is seen as a sign of liveliness. When a peer is 
declared dead "the connection" should be closed. More correct phrasing 
would be indeed "the channel should be closed", which implies no more 
messages will be sent to the peer and the local administration about the 
peer is cleared. We will change "connection" to "channel".
Does this answer your question?


> Q2: As the sending of keep alives is a SHOULD, are there example
> cases when keep alives would NOT be sent?
>

One example is that of a busy server that hosts complete copies of 
static content. In that case, the server could refrain from sending keep 
alives to save on processing and outgoing messages. In particular,
assume a client establishes a channel with the server, sends a REQUEST, 
and receives a DATA message in reply. The client acknowledges it with an 
ACK, but after that does not attempt any downloads, and just sends keep 
alives.

Ideally, this client should be garbage collected. The server can achieve 
that by not sending keep alives after the DATA message. After 3 minutes, 
the client will conclude that the server is dead and discard
the channel. And after three minutes, the server can clean up his end.
If the client would send new REQUESTs in those three minutes, the 
cleanup would be avoided.


> Q3: The text saying "to each peer it wants to interact with in the
> future" sounds a little strange to me. How does a peer know with whom
> it wants to interact in the future? Perhaps the text instead should
> talk about peers with whom one wants to maintain a signaling channel,
> or something like that?
>

It is indeed about maintaining a signaling channel. Most P2P systems 
have a policy that decides which peers are of interest now and in the 
near future. Examples of interesting peers are peers that have chunks on 
offer that this client needs, or peers that currently do not have 
interesting chunks on offer (because they are still downloading 
themselves, or in live streaming), but gave good performance in the 
past. When they have new stuff to offer they can be used again,
so you want to keep a signaling channel open.

 From a subset of those peers the client will request chunks. In P2P
systems that have tit-for-tat as freeriding prevention this is normally
a small subset as to not fragment the upload bandwidth needed to
reciprocate the peer.

Is it the phrasing that raises your question or should we explain more 
about this policy for deciding which peers are interesting?

Regards,
       Arno Bakker et al.