orientjs icon indicating copy to clipboard operation
orientjs copied to clipboard

"Maximum call stack size exceeded" trigerred by remote OrientDB exception

Open jbilcke opened this issue 8 years ago • 2 comments

On OrientJS 2.2.6 + OrientDB 3.0 (which actually works fine with orientjs!)

I got this infinite recursion error in the client, triggered by a server error :

On RangeError: Maximum call stack size exceeded
    at Buffer.Uint8Array (native)
    at FastBuffer (buffer.js:11:5)
    at allocate (buffer.js:184:13)
    at Function.Buffer.allocUnsafe (buffer.js:152:10)
    at new Buffer (buffer.js:83:19)
    at Connection.handleSocketData (node_modules/orientjs/lib/transport/binary/connection.js:292:14)
    at Connection.handleSocketData (node_modules/orientjs/lib/transport/binary/connection.js:311:12)
    at Connection.handleSocketData (node_modules/orientjs/lib/transport/binary/connection.js:311:12)
    at Connection.handleSocketData (node_modules/orientjs/lib/transport/binary/connection.js:311:12)
   ...
   ...

This was caused by an exception on the remote database side, which I fixed after going through the db server log (I was trying to run a query relying on a a vertex class that did not exist yet, causing an OCommandExecutionException)

But still, it seems a bit dangerous to have such infinite recursion possible in the JS client, perhaps it could be improved?

edit: I'm not sure how it relates to #272 as I do not have as many entries in my database (0, actually). Also, I think we use different versions (the line numbers in the stack traces are different)

jbilcke avatar Aug 12 '17 12:08 jbilcke

hi @jbilcke

yes orientjs 2.2.6 works with OrientDB 3.0 since it will use the 2.2.x protocol.

this recursion probably is due to parsing error from the server.

How can i reproduce this?

Are you able to reproduce this 2.2.x?

wolf4ood avatar Aug 12 '17 12:08 wolf4ood

Hi @jbilcke

we just started shipping OrientJS 3.0 with the complete support of new OrientDB 3.0 features. It's still in beta

here it's the release

and there it's the documentation

wolf4ood avatar Jul 03 '18 10:07 wolf4ood