Skip to content

Conversation

@rogersm
Copy link

@rogersm rogersm commented Dec 28, 2011

Bill,

I just downloaded it, but limited-thread-taskmasker expects to use hunchentoot:log-message instead of hunchentoot:log-message*

I updated the code to use it.

@billstclair
Copy link
Owner

hunchentoot:log-message* isn't in Hunchentoot 1.1.1. Is it in a later
version, renamed from hunchentoot:log-message? If so, then I'll have
to support both. Also, if so, good to know that
limited-thread-taskmaster still works except for that. I believe that
newer versions of Hunchentoot have their own way of limiting the
number of threads, but, IIRC, it doesn't queue up requests, instead it
returns an error to the client.

-Bill

On Wed, Dec 28, 2011 at 2:01 PM, rogersm
[email protected]
wrote:

Bill,

 I just downloaded it, but limited-thread-taskmasker expects to use hunchentoot:log-message instead of hunchentoot:log-message*

 I updated the code to use it.

You can merge this Pull Request by running:

 git pull https://github.com/rogersm/limited-thread-taskmaster master

Or you can view, comment on it, or merge it online at:

 #1

-- Commit Summary --

  • Updated hunchentoot:log-message to hunchentoot:log-message*

-- File Changes --

M limited-thread-taskmaster.asd (2)
M src/taskmaster.lisp (4)

-- Patch Links --

 https://github.com/billstclair/limited-thread-taskmaster/pull/1.patch
 https://github.com/billstclair/limited-thread-taskmaster/pull/1.diff


Reply to this email directly or view it on GitHub:
#1

@rogersm
Copy link
Author

rogersm commented Dec 29, 2011

Bill,

I had no idea the API had changed. I was doing some prototyping with a newer version of hunch and I discovered this issue.
And you're right with hunchentoot changes: the newer releases have improved and currently limit and queue requests.

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