Jan

Results 157 comments of Jan

@lshaf : you mean like this? ``` index 3e1f0b5..a0e5eea 100644 --- a/lib/ArangoDBClient/EdgeHandler.php +++ b/lib/ArangoDBClient/EdgeHandler.php @@ -74,7 +74,9 @@ class EdgeHandler extends DocumentHandler */ public function createFromArrayWithContext($data, $options) { - return...

Looking at this again, I think the main problem with using $_documentClass everywhere is that there may be different document classes for documents and edges. For example, the following code...

Example implementation in https://github.com/arangodb/arangodb-php/pull/251 @lshaf @diabl0 : can you please comment on it? Thanks!

@sandrokeil: this is a good idea and we also thought about that a bit. From my experience with the native driver for node.js I would expect that the performance gains...

Not sure if I understand the problem entirely, but the padding in general solves the following problem when building VelocyPack objects/arrays programmatically: ``` using namespace arangodb::velocypack; Builder b; b.openArray(); //...

WDYT about making the padding more controllable in the Builder, e.g. via https://github.com/arangodb/velocypack/pull/39?

I see. I have pushed another commit (https://github.com/arangodb/velocypack/pull/39/commits/ce824eead5fc12ae47a3b37964cadb7832219084) that will dismiss Array types 0x02, 0x03 and 0x04 in favor of using type 0x05 when padding is explicitly requested. That should...

Yes, our implementation will recognize the 0x02, 0x03, 0x04 and 0x05 formats. I will add yet another validator test to ensure this is also test-covered there. Regarding generating the "new"...

I haven't made up my mind about that yet. I am happy to think more about that, but it would help if you could either send or point to some...

On the one hand, building a full Doctrine ODM for ArangoDB seems pretty complex. Furthermore, not all projects out there use Doctrine so there should be some alternative for these....