[
https://issues.apache.org/jira/browse/TS-3535?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15208432#comment-15208432
]
ASF GitHub Bot commented on TS-3535:
------------------------------------
Github user zwoop commented on the pull request:
https://github.com/apache/trafficserver/pull/525#issuecomment-200354234
+1 on adding it to lib/ts. As for STL, the general rule is that if
something runs on the critical path (e.g. as part of the HttpSM, or normal
transaction handling), where it is performance sensitive, we should make every
effort to avoid STL. As a subsidiary to that, we should really think hard
before using std::string on said critical path as well (there's generally
little reason to use std::string there).
Now, the exception here are two things:
1) Plugins. We don't dictate anything for plugins, so use whatever you deem
necessary to make it work. If someone doesn't like it, they can either fix it,
or they can not use the plugin.
2) Anything not on the critical path. This includes configuration loading,
traffic_manager/cop, periodic tasks, tests etc.
That much said, don't use STL just for the hell of it. If there is a
reasonable alternative to use in lib/ts, please use it. Dragging in STL does
increase binary sizes, which could affect performance in other ways (such as
worse L2/3 caches etc.). It also sets bad precedence, where someone seeing it
used, makes the mistake and thinks it's ok to use everywhere.
> Add priority feature to the HTTP/2 implementation
> -------------------------------------------------
>
> Key: TS-3535
> URL: https://issues.apache.org/jira/browse/TS-3535
> Project: Traffic Server
> Issue Type: Improvement
> Components: HTTP/2
> Reporter: Bryan Call
> Assignee: Masaori Koshiba
> Fix For: 6.2.0
>
>
> Prioritizes the responses back to the client based on the priority level
> specified by the HTTP/2 protocol.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
