Skip to content

Enable DRPC request to be sent via HTTP and/or Thrift - #733

Open
anfeng wants to merge 11 commits into
nathanmarz:masterfrom
anfeng:http-drpc
Open

Enable DRPC request to be sent via HTTP and/or Thrift#733
anfeng wants to merge 11 commits into
nathanmarz:masterfrom
anfeng:http-drpc

Conversation

@anfeng

@anfeng anfeng commented Nov 8, 2013

Copy link
Copy Markdown
Collaborator

Currently, DRPC requests could only be sent via Thrift API. We have seen various users asking for HTTP interface.

This pull request enable one to configure drpc to be sent via HTTP and/or thrift via storm.yaml:
drpc.port: <THRIFT_PORT> (default: 3772)
drpc.http.port: <HTTP_PORT> (default: unavailable)

When drpc server is started, it will look into these configuration parameters to decide whether Thrift port and/or HTTP port should be binded.

DRPC HTTP request will be received via GET via the following URI:

  • http://<server>:<HTTP_PORT>/drpc/<Func>/<Args>
  • http://<server>:<HTTP_PORT>/drpc/<Func>/
  • http://<server>:<HTTP_PORT>/drpc/<Func>

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this is in here twice by accident.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Oops I missed the trailing '/'. Looks like jetty is not handling '/' characters very well.

@revans2

revans2 commented Nov 8, 2013

Copy link
Copy Markdown
Contributor

It looks good.

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.

2 participants