Difference: RegistryInterface101RFC (10 vs. 11)

Revision 112008-12-04 - TomMcGlynn

 
META TOPICPARENT name="RegistryInterface"

Registry Interface Proposed Recommendation: Request for Comments

This document will act as RFC center for the Proposed Recommendation entitled "IVOA Registry Interfaces, version 1.01". The specification can be found at http://www.ivoa.net/Documents/cover/RegistryInterface-20080929.html (PDF, DOC).

Review period: 30 September 2008 to 29 November 2008*
    *extended from original end date.

Note on ADQL Reference:
The reference for ADQL in the Registry Interfaces document is incorrect. The actual URL should be http://www.ivoa.net/Documents/WD/ADQL/ADQL-20050624.pdf.

In order to add a comment to the document, please edit this page and add your comment to the list below in the format used for the example (include your WikiName so authors can contact you for further information). When the author(s) of the document have considered the comment, they will provide a response after the comment.

Additional discussion about any of the comments or responses can be conducted on the Resource Registry mailing list, registry@ivoa.net. However, please be sure to enter your initial comments here for full consideration in any future revisions of this document.

Comments from the Community

NoelWinstanley 7 Oct 2008

  • The Search operation requires the query to be in ADQL/x format. However, recent versions of the ADQL specification (including the PR) makes no mention of ADQL/x, only the ADQL/s form. The reference given by the Registry Interface document for ADQL/x is to a 2004 working draft, which must have been superceded by the ADQL PR (the URL is broken, BTW).
    • Is it sensible to base the main querying interface to registry on a 'dead' language?
    • Or is KeywordSearch the function that's recommended to be actually used in practice?
    • Should the Search function be marked as deprecated / liable to change ?
    • Should the Search function be marked as optional, and support described in the registry, as for XQuerySearch?
    • Should the 'Search' function be altered to accept adql/s instead?

Is it sensible to base the main querying interface to registry on a 'dead' language?
This is a good question that I would like to hear more comments on and which the TCG should specifically take up. While not ideal, our rational is that as this specification contains a number of other components that are critical to registry interoperability, we thought it important not to slow the document by trying to upgrade the ADQL part (which has an impact on implementations). It was intended that RI be tied specifically to ADQL v1.0 so as not to be confused with the later ADQL revamp; however, I see that the body of the text is not explicit and the reference is incorrect; only the WSDL is correct. Making the ADQL interface optional like XQuerySearch may be a reasonable alternative.

Additional Note: As suggested and endorsed at the October Interop, we will inline the relevent bits of ADQL v1.01 into the RI spec. This may not get added in until after the RFC; see the ADQL spec until then.


Comments by TCG

Chairs should add their comments under their name.

Data Curation and Preservation (BobHanisch)

I defer to the registry specialists on this one. The document certainly looks complete and well thought out.

The document is correctly labeled as Proposed Recommendation, but the text in the paragraph Status of the Document still says Working Draft.

Under Conformance-Related Definitions, "shall" is defined as part of the restricted vocabulary, but it is not actually used in the document. I suggest removing "shall" from this list. The IETF RFC 2119 equates "shall" with "must" in any case, so we might as well use just the one term.

AstroRG (MasatoshiOhishi)

I appreciate the hard work made by the Registry WG members and its collaborators, and I trust the outcome by these expterts.

TCG (ChristopheArviset)

I trust the specialists on these issues so I basically agree with the document, but I have a few comments/questions for clarification.

  • page numbers should be added on each page
  • section 2.1.2, the ADQL schema referenced is 1.0. Is that correct or should it be 2.0 ?
  • appendix 4 : should we also add new prefixes such a VOSpace? As a more general question, how do we add more prefix in the future when more IVOA standards get approved (eg TAP, SLAP) ? Does that mean we need to update this document each time ? Some clarification would be useful on this process in the document.
  • references : ADQL reference should be updated and STC reference should be added
Added:
>
>

Applications (Tom McGlynn)

I've got lots of comments but few of them are really substantive, they are mostly requests for clarification.

Document says it's a working draft, but it is actually a Proposed Recommendation -- that seems to be a general problem.

1.1 I don't believe that it is ever clearly stated that the functional distinction between a publishing registry and a searchable registry is that a publishing registry does not support the search functions of section 2. The statement is made that "it does not need to support...". I think a clearer statement would be something like:

Only searchable registries support the protocols discussed in section 2. All VO compliant registries must support the protocols discussed in section 3.

2.1 I don't see what advantage there is in making the distinction between search and resolve operations.

Note that getResource presumably returns 0 resources if the ID is not known (i.e., it is not "one and only one")

2.1.1 It seems to me that the error response should be something that is standardized at a higher level than a specific protocol. Too late for that though I suppose.

It was unclear to me if the paging of results is done purely at the request of the user or can be initiated by the server if some constraint is exceeded (as in the OAI interface). Could this be clarified.

2.1.2 the "hitherto referred to using the "adlq:" prefix" doesn't make sense, since as far as I can tell it was never used up to that point. Do you mean "from now on..." --- that's the opposite of hitherto?

I don't like the idea of a required attribute being set to a null value with Table. Personally I'd prefer some string, e.g., "Registry".

I have no idea what the adql:Column elements look like and I think you should give an example here, or replicate the appropriate item in the schema if it's short enough.

Why must there be an implicit addition of the status requirement? This is redundant with a statement in 2.1.1 I think. [Personally I think this is not necessarily a good idea. What if I want to know how many records in a registry have been deleted. These is no way to do this using the search interface even though it is a trivial query to formulate. Maybe some kind of flag allowing old records shoudl be available.]

2.1.2.1 is very confusing to me. Move the examples up and put them in the text rather than a box even if they are non-normative.

The first note in 2.1.2.1 is mystifying to me. I'd make it a footnote.

In the note about harvesting... I'm not sure what this means? Am I forbidden to harvest using these methods? I don't think so -- I certainly hope not. We don't tell users what they cannot do. We tell them what they can. I think this note is meaningless. We can recommend use of the OAI protocols for harvesting but that's as far as we should go.

The last note in this section -- which seems to be mostly notes! -- is also somewhat mysterious. I think the issues raised in the first and last notes should be discussed more clearly in a non-normative appendix.

2.1.3 I'm concerned about white space. If the registry contains a field which has two consecutive blanks then I gather that I cannot match against it since consecutive white space on the input is to be compressed to a single white space.

The document suggests that all white-space characters in the search field match all white-space characters in the registry text. E.g., space in the input matches a carriage return in the text.

How about just saying something like: The keyword value is a string where leading and trailing white space is discarded. The string is broken into keyword tokens using white space delimiters except when the white space is enclosed in quote(e.g. ...). The characters ... are considered white space. A token that begins and ends with "'s has them removed.

and leave off any mention of compression.

By the by, is there an escape mechanism for quotes? Currently I don't think I can look for the string " can" versus "can" or for "can not" versus "can not". I don't see why these are explicitly precluded.

I'd put the note in the main text.

2.2.1

Personally I'd just reuse the output format from the search.

2.2.2

If "Not found" is thought to be an error, it should give an error response. If it isn't an error, then don't say an "error message" should be added. The use of the "fault" seems to be an effort to sidestep the word error but makes everything less clear to me.

Personally I'd suggest treating NOT FOUND as a perfectly natural response to GetResource but I think the corresponding action for getIdentity is an error.

2.2.3 I suspect "hitherto" is again being misused -- though "vg:" does appear once previously.

3.1.1 Is the last item in the list of OAI-PMH useful features "deleted records not returned" a requirement of OAI? I'm not sure if the HEASARC is compliant if so.

3.1.2 What is the reason for the "strongly recommended"? It seems to me that this relates to one of the notes in 2.1. I don't think there is any status called strongly recommended. There is MUST and SHOULD. Require it if it's really necessary.

3.1.3 Make the note a footnote. It's irrelevant to use within the VO.

3.2 I believe the discussion of the validation level belongs in the text not in notes. Perhaps a whole section to itself!

 


<--  
-->
 
This site is powered by the TWiki collaboration platform Powered by Perl This site is powered by the TWiki collaboration platformCopyright © 2008-2024 by the contributing authors. All material on this collaboration platform is the property of the contributing authors.
Ideas, requests, problems regarding TWiki? Send feedback