Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Apify.call() is missing an option to set up memory #98

Closed
metalwarrior665 opened this issue May 15, 2018 · 3 comments
Closed

Apify.call() is missing an option to set up memory #98

metalwarrior665 opened this issue May 15, 2018 · 3 comments
Assignees
@mtrunkat
Labels
feature Issues that represent new features or improvements to existing features.

Comments

@metalwarrior665
Copy link
Member

No description provided.

@mtrunkat mtrunkat added the feature Issues that represent new features or improvements to existing features. label Jun 7, 2018
@metalwarrior665
Copy link
Member Author

I would like to add this, it seems straightforward. It is in the client.

The thing is if we want to somehow control the memory to be 2 to the power of something. Right now in API it is not controlled at all, you can run the act with memory 1234.

@mtrunkat
Copy link
Member

We let use enter any value in SDK/client. If the memory won't be a power of 2 then server throws an error which appears in actor log. I think that this is better behavior than having some validation in SDK, client and server.

@mtrunkat mtrunkat self-assigned this Jun 22, 2018
@mtrunkat
Copy link
Member

Fixed in new version that will be shortly released as BETA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mtrunkat mtrunkat

Labels
feature Issues that represent new features or improvements to existing features.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mtrunkat @metalwarrior665