[apps-discuss] Schema comments on draft-ietf-mile-sci (fwd)

Claudio Allocchio <Claudio.Allocchio@garr.it> Wed, 22 January 2014 09:34 UTC

Return-Path: <Claudio.Allocchio@garr.it>
X-Original-To: apps-discuss@ietfa.amsl.com
Delivered-To: apps-discuss@ietfa.amsl.com
Received: from localhost (ietfa.amsl.com [127.0.0.1]) by ietfa.amsl.com (Postfix) with ESMTP id 67BAB1A0099 for <apps-discuss@ietfa.amsl.com>; Wed, 22 Jan 2014 01:34:12 -0800 (PST)
X-Virus-Scanned: amavisd-new at amsl.com
X-Spam-Flag: NO
X-Spam-Score: 0.544
X-Spam-Level:
X-Spam-Status: No, score=0.544 tagged_above=-999 required=5 tests=[BAYES_00=-1.9, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, HELO_EQ_IT=0.635, HOST_EQ_IT=1.245, J_CHICKENPOX_35=0.6, J_CHICKENPOX_36=0.6, RP_MATCHES_RCVD=-0.535, SPF_PASS=-0.001] autolearn=no
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 afLRJRLuBXAz for <apps-discuss@ietfa.amsl.com>; Wed, 22 Jan 2014 01:34:11 -0800 (PST)
Received: from cyrus.dir.garr.it (cyrus.dir.garr.it [193.206.158.29]) by ietfa.amsl.com (Postfix) with ESMTP id 3F61F1A007D for <apps-discuss@ietf.org>; Wed, 22 Jan 2014 01:34:11 -0800 (PST)
Received: internal info suppressed
Date: Wed, 22 Jan 2014 10:34:05 +0100
From: Claudio Allocchio <Claudio.Allocchio@garr.it>
X-X-Sender: claudio@mac-allocchio3.garrtest.units.it
To: apps-discuss@ietf.org
Message-ID: <alpine.OSX.2.02.1401221033270.38263@mac-allocchio3.garrtest.units.it>
User-Agent: Alpine 2.02 (OSX 1266 2009-07-14)
MIME-Version: 1.0
Content-Type: TEXT/PLAIN; charset="US-ASCII"; format="flowed"
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=garr.it; s=cyrus; t=1390383247; bh=3jdXwa9ls+we8ueRxlxEAkncxjpztfAQtvqRXd5DGr0=; h=Date:From:To:Subject; b=YWdrJH/UxgU3mq/88gdKBg5J9jEuvH5YexCrJ605CoJ2KUbx4a5dKF+8cWRu52mfP VA4W2tS3QZPLooikw7DykxwxkMUDHGWpfl4I71KFRxAM4auuDGrLro/xJMsBCK//JB 4q6MMLznbPzeZbXE29D+5TCJj295ZV7r7/tfNS4Q=
Subject: [apps-discuss] Schema comments on draft-ietf-mile-sci (fwd)
X-BeenThere: apps-discuss@ietf.org
X-Mailman-Version: 2.1.15
Precedence: list
List-Id: General discussion of application-layer protocols <apps-discuss.ietf.org>
List-Unsubscribe: <https://www.ietf.org/mailman/options/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=unsubscribe>
List-Archive: <http://www.ietf.org/mail-archive/web/apps-discuss/>
List-Post: <mailto:apps-discuss@ietf.org>
List-Help: <mailto:apps-discuss-request@ietf.org?subject=help>
List-Subscribe: <https://www.ietf.org/mailman/listinfo/apps-discuss>, <mailto:apps-discuss-request@ietf.org?subject=subscribe>
X-List-Received-Date: Wed, 22 Jan 2014 09:34:12 -0000

---------- Forwarded message ----------
Date: Wed, 22 Jan 2014 07:23:17 +0100
From: Martin Thomson <martin.thomson@gmail.com>
To: draft-ietf-mile-sci.authors@tools.ietf.org,
     draft-ietf-mile-sci.chairs@tools.ietf.org,
     Claudio Allocchio <Claudio.Allocchio@garr.it>
Subject: Schema comments on draft-ietf-mile-sci

I did take a quick look at the schema.  This is a fairly big, yet simple 
schema.

At this stage, the only comments I'd be prepared to advance :

There is no need to wrap <xs:choice> with <xs:sequence> when both have
cardinality 1.  That's redundant.

schemaLocation on the import is unnecessary (and not especially useful
given that it's a URN).

use of xsd:string throughout is an anti-pattern.  xsd:token is better,
even more so if you can restrict the value-space further;
unconstrained strings can be a real burden later.  In particular,
xsd:string causes usability issues, and can lead to some major issues
with whitespace normalization.  Unless you need to carefully preserve
whitespace, I'd strongly recommend against its use.