TIMEOUT keyword in SQL queries

[yojimbo87]

SQL queries should have a TIMEOUT keyword where represents time in miliseconds that indicates maximum time until the query should return the entire result.

Discussion:
https://groups.google.com/forum/#!topic/orient-database/7vKwvjFoNyE

[lvca]

Done. I'm testing it right now. I've added the new syntax:

TIMEOUT <ms> [<strategy>]

Where ms is the timeout. 0 (zero) means no timeout. the strategy, optional can be:

• exception (the default), throws a OTimeoutException when the timeout is reached
• return, just return the collected result so far

Example:

select from V timeout 1 return

It's the best way to see how many records can fetch in 1 ms. If you execute it multiple time you should see different values with so small timeout.

By default the timeout is taken by global parameter (default=0, no timeout):

COMMAND_TIMEOUT("command.timeout", "Default timeout for commands expressed in milliseconds", Long.class, 0),

If the client declare a timeout > then the server's default, then the server timeout wins.

[Volune]

I was trying the node driver with the last server revision, but I had a deserialization error on server side.

At OCommandRequestTextAbstract.java#L186

    timeoutMs = buffer.getAsLong();
    timeoutStrategy = TIMEOUT_STRATEGY.values()[buffer.getAsByte()];

When streaming a request, is the timeout now mandatory for any kind of request, or is this a mistake ?
These lines seemed strange because there already is the support of the TIMEOUT keyword in the SQL queries.

[lvca]

Done!

[Volune]

Thanks