Skip to content

TS-4888: collapsed_forwarding plugin should return TSREMAP_NO_REMAP.#1042

Merged
jpeach merged 1 commit intoapache:masterfrom
brkishore:TS-4888
Sep 26, 2016
Merged

TS-4888: collapsed_forwarding plugin should return TSREMAP_NO_REMAP.#1042
jpeach merged 1 commit intoapache:masterfrom
brkishore:TS-4888

Conversation

@brkishore
Copy link
Copy Markdown

Modified collapsed_forwarding plugin to return TSREMAP_DID_REMAP. And also created TSRemapDeleteInstance() to avoid the runtime/loading error.

@jpeach
Copy link
Copy Markdown
Contributor

jpeach commented Sep 23, 2016

[approve ci]

@atsci
Copy link
Copy Markdown

atsci commented Sep 23, 2016

FreeBSD build successful! See https://ci.trafficserver.apache.org/job/Github-FreeBSD/859/ for details.

@atsci
Copy link
Copy Markdown

atsci commented Sep 23, 2016

Linux build successful! See https://ci.trafficserver.apache.org/job/Github-Linux/755/ for details.

@zwoop zwoop added the Plugins label Sep 23, 2016
@zwoop zwoop added this to the 7.1.0 milestone Sep 23, 2016
@zwoop zwoop self-assigned this Sep 23, 2016
Copy link
Copy Markdown
Contributor

@zwoop zwoop left a comment

Choose a reason for hiding this comment

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

Seems good.

@pbchou
Copy link
Copy Markdown
Contributor

pbchou commented Sep 23, 2016

Take care when merging, the commit comment should say return TSREMAP_NO_REMAP.

@brkishore brkishore changed the title TS-4888: Modified collapsed_frowarding plugin to return TSREMAP_DID_R… TS-4888: Modified collapsed_frowarding plugin to return TSREMAP_NO_R… Sep 26, 2016
@jpeach jpeach changed the title TS-4888: Modified collapsed_frowarding plugin to return TSREMAP_NO_R… TS-4888: collapsed_forwarding plugin should return TSREMAP_NO_REMAP. Sep 26, 2016
@jpeach jpeach merged commit 86aa594 into apache:master Sep 26, 2016
bneradt pushed a commit to bneradt/trafficserver that referenced this pull request Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants