Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make references normative #4179

Closed
wants to merge 1 commit into from
Closed

Make references normative #4179

wants to merge 1 commit into from

Conversation

janaiyengar
Copy link
Contributor

Closes #4178.

Copy link
Contributor

@ianswett ianswett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not the expert on what should be normative, but these were intentionally marked as informative references since they're all TCP specific RFCs. The pipeACK reference in particular seems like an odd one.

If this is what the chairs want, then I'm ok with it as an editor, but these choices were intentional.

Copy link
Contributor

@martinduke martinduke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see what @gloinul is saying, but I also see some wiggle room in RFC 3967, Section 1.1:

  • When a reference to an example is made, such a reference need not
    be normative. For example, text such as "an algorithm such as the
    one specified in [RFCxxxx] would be acceptable" indicates an
    informative reference, since that cited algorithm is just one of
    several possible algorithms that could be used.

I am also somewhat concerned that trying to get the downrefs into the registry is more likely to run into trouble (Sec 3)

This procedure should not be used if the proper step is to move the
document to which the reference is being made into the appropriate
category. It is not intended as an easy way out of normal process.
Rather, the procedure is intended for dealing with specific cases
where putting particular documents into the required category is
problematic and unlikely ever to happen.

Anyway, if we decide to proceed the PR is fine modulo my comment below.

@@ -1133,8 +1133,8 @@ limits and so no advantage is gained by doing so.

Endpoints choose the congestion controller that they use. Congestion controllers
respond to reports of ECN-CE by reducing their rate, but the response may vary.
Markings can be treated as equivalent to loss ({{?RFC3168}}), but other
responses can be specified, such as ({{?RFC8511}}) or ({{?RFC8311}}).
Markings can be treated as equivalent to loss ({{!RFC3168}}), but other
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This section doesn't have normative language, so I don't see a need for these references to be normative.

Copy link
Member

@martinthomson martinthomson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think that any of these need to be made normative either. Each of these is exemplary only.

Copy link
Contributor

@ianswett ianswett left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing my approval, since I don't really want to make this change and it appears others have similar opinions and letting the conversation on the issue play out.

@janaiyengar
Copy link
Contributor Author

Closing in favor of #4190

@martinthomson martinthomson deleted the jri/normative branch November 5, 2020 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Recovery-31: Normative References
4 participants