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
Merge server flags #53
Conversation
|
a68fdab
to
48ac0b9
Compare
app/src/entry-point.lisp
Outdated
|
||
;; configure the server | ||
(unless (or port start-web-server) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
start-web-server
is not a variable
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch @notmgsk
|
||
;; launch the polling loop | ||
(show-banner) | ||
(format t "~% | ||
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> IMPORTANT NOTICE <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<< | ||
The HTTP endpoint has been deprecated in favor of the RPCQ endpoint. In the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we note that an RPCQ server is being started
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The actual console output (sorta) indicates that. Is what's there insufficient?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here's the sort of thing that gets emitted:
boson:quilc eric$ ./quilc -S -p 6000
+-----------------+
| W E L C O M E |
| T O T H E |
| R I G E T T I |
| Q U I L |
| C O M P I L E R |
+-----------------+
Copyright (c) 2016-2019 Rigetti Computing.
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> IMPORTANT NOTICE <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
The HTTP endpoint has been deprecated in favor of the RPCQ endpoint. In the
future, it will be removed. You're advised to modify your client code to talk
to the RPCQ version instead.
>>>>>>>>>>>>>>>>>>>>>>>>>>>>> END IMPORTANT NOTICE <<<<<<<<<<<<<<<<<<<<<<<<<<<<<
WARNING: -p and -S are incompatible. Dropping -p.
<134>1 2019-02-01T19:30:28Z boson.lan ./quilc 5649 LOG0001 - Launching quilc.
<134>1 2019-02-01T19:30:28Z boson.lan ./quilc 5649 - - Spawning server at (tcp://*:5555) .
[2019-02-01 14:30:28] Starting server: 0.0.0.0 : 6000.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
IMO it's not clear. I think I might be confused if I saw that two servers were being spawned when I only requested one.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll add some more text to the IMPORTANT NOTICE.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
>>>>>>>>>>>>>>>>>>>>>>>>>>>>>>> IMPORTANT NOTICE <<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<
The HTTP endpoint has been deprecated in favor of the RPCQ endpoint. In the
future, it will be removed. In the meanwhile, we are launching *both* an HTTP
server and an RPCQ server. You're advised to modify your client code to talk
to the RPCQ version instead, so that it continues to operate when we disable the
HTTP server for good.
>>>>>>>>>>>>>>>>>>>>>>>>>>>>> END IMPORTANT NOTICE <<<<<<<<<<<<<<<<<<<<<<<<<<<<<
48ac0b9
to
7a823b4
Compare
This PR makes the transition from the HTTP server to the RPCQ server more painless: when quilc is launched with
-S
, it launches the RPCQ server in addition to the web server, so that modern pyQuil is about to quietly use the new endpoint.