Open dthain opened 11 months ago
@tphung3 please take a crack at this one. In the current state, I think serverless is working for functions that don't violate any of the constraints above, but you should do some testing of your own. These mixups should make the system much more robust.
@JinZhou5042 fixed bullet 5 in #3764
(Note this is relative to PR #3436 that hasn't been merged yet.)
The protocol between the worker and the serverless library has several infelicities that require a coordinated fix between the library code and
vine_process_invoke_function
VINE_LINE_MAX
. It should instead be written as an integer length followed by the actual data. (Similar to that of the function input.)vine_process_invoke_function
assumes that the input is text and gets the length withstrlen
. Measure the size of the file instead so that we can send arbitrary data.