Hi Qiao, Thanks for your review. Authors will consider your comments and update the document soon. About the technical issues you mentioned, see my comments inline:
Thanks, Jensen On Sun, Jul 8, 2018 at 11:16 AM Qiao Xiang <[email protected]> wrote: > Dear Sabine, Jensen and Dawn, > > I finished my review on Section 1-6 of the Unified Property draft. In the > attached file,, all the comments are marked wtih "[qiao]". I will send > out the reviews for the remaining sections soon. > > Most of my comments are related to consistency and clarity. However, there > are two technical issues I want to emphasize here, to seek the opinions > from you and other WG members: > > 1. In Section 4.4: in the PropertyMapCapabilities object to define the > capabilities of a Property Map: the types of entity domains and the types > of properties provided in this map are each defined using a list, > respectively. With such a spec, how should the client know which domain has > which property? Consider the case where the server announces that it has a > property map with two entity domains d1 and d2, and property p1 and p2. If > the client wants to know the property of some entities in domain d1, it > sends a filtered property map query to the server. But it turns out, the > server only provides property p1 for domain d2. As such, the client gets a > bunch of "null" or error code. Only by now the client knows that the server > only provides property p1 for domain d2 entities, but a round-trip is > wasted. Simple as this example is, it can turn into a much troubling > scenario: imaging X (e.g., 10) domains and Y (e.g., 20) properties. > > One solution to fix this issue, proposed during my offline discussion with > Jensen, is to redefine the PropertyMapCapabilities using an array of > (entity, property) combination. > > Sure, we already discussed offline. But after that, I rethink about this problem. In a single property map resource, the "prop-types" and the "entity-domain-types" should be fully bound. So if an ALTO server wants to provide property p1 and p2 for domain d1, and only property p1 for domain d2, the best practice should be to separate domain d1 and d2 to different property maps. Does it make sense? Waiting for more comments from others. > 2. In Section 6.3, the writing seems to only focus on the pid property of > IP entities. But conceptually other entity domains may also have a "pid" > property, right? I understand that this section is mainly to discuss the > compatibility with the pid property provided by ALTO EPS service, but it > should be stated clearly so that the readers would not be confused. > > Although we do not mention that "pid" property must bind with the IP entities, I'm not sure whether we should emphasize this. Because in this document, we only define IP domain and PID domain. And only IP domain may own the "pid" property. If the future documents define another entity domain owning a property called "pid", it can specify the semantics by itself. > Please let me know if you have any thoughts on these. Thanks. > > > Best > Qiao > > > > > > On Fri, Jun 29, 2018 at 2:11 PM Randriamasy, Sabine (Nokia - > FR/Paris-Saclay) <[email protected]> wrote: > >> Hello, >> >> The main updates in this new version relate to the consistency procedure >> between ALTO Address Type Registry and >> ALTO Entity Domain Registry as discussed during the ALTO WG session in >> London. This is addressed in section 9. IANA Considerations, where 9.2 >> specifies the ALTO Entity Domain Registry and 9.2.1. proposes an algorithm >> to ensure consistency between both registries. >> >> Other updates include: >> - in section 1. Introduction: a paragraph introducing ALTO Entity domains, >> - In section 6.3, some rewording to clarify between "pid" and "PID" to >> avoid headaches, >> - section 7.3 example IRD: name update for the Endpoint property resource >> - section 7.4: example transaction >> - Section 10 References: updates and reformatting >> - usage of expression "ALTO Entity Domain" throughout the document >> >> Your feeback will be highly appreciated >> >> Thanks >> Sabine, Jensen, Dawn >> >> -----Original Message----- >> From: alto [mailto:[email protected]] On Behalf Of >> [email protected] >> Sent: Friday, June 29, 2018 7:29 PM >> To: [email protected] >> Cc: [email protected] >> Subject: [alto] I-D Action: draft-ietf-alto-unified-props-new-04.txt >> >> >> A New Internet-Draft is available from the on-line Internet-Drafts >> directories. >> This draft is a work item of the Application-Layer Traffic Optimization >> WG of the IETF. >> >> Title : Unified Properties for the ALTO Protocol >> Authors : Wendy Roome >> Shiwei Dawn Chen >> Sabine Randriamasy >> Y. Richard Yang >> Jingxuan Jensen Zhang >> Filename : draft-ietf-alto-unified-props-new-04.txt >> Pages : 30 >> Date : 2018-06-29 >> >> Abstract: >> This document extends the Application-Layer Traffic Optimization >> (ALTO) Protocol [RFC7285] by generalizing the concept of "endpoint >> properties" to domains of other entities, and by presenting those >> properties as maps, similar to the network and cost maps in ALTO. >> >> >> >> The IETF datatracker status page for this draft is: >> https://datatracker.ietf.org/doc/draft-ietf-alto-unified-props-new/ >> >> There are also htmlized versions available at: >> https://tools.ietf.org/html/draft-ietf-alto-unified-props-new-04 >> https://datatracker.ietf.org/doc/html/draft-ietf-alto-unified-props-new-04 >> >> A diff from the previous version is available at: >> https://www.ietf.org/rfcdiff?url2=draft-ietf-alto-unified-props-new-04 >> >> >> Please note that it may take a couple of minutes from the time of >> submission until the htmlized version and diff are available at >> tools.ietf.org. >> >> Internet-Drafts are also available by anonymous FTP at: >> ftp://ftp.ietf.org/internet-drafts/ >> >> _______________________________________________ >> alto mailing list >> [email protected] >> https://www.ietf.org/mailman/listinfo/alto >> >> _______________________________________________ >> alto mailing list >> [email protected] >> https://www.ietf.org/mailman/listinfo/alto >> > > > -- > Qiao Xiang > Postdoctoral Fellow, > Department of Computer Science, > Yale University > _______________________________________________ > alto mailing list > [email protected] > https://www.ietf.org/mailman/listinfo/alto >
_______________________________________________ alto mailing list [email protected] https://www.ietf.org/mailman/listinfo/alto
