bchavez / RethinkDb.Driver

:headphones: A NoSQL C#/.NET RethinkDB database driver with 100% ReQL API coverage.
http://rethinkdb.com/api/java
Other
384 stars 134 forks source link

OutOfMemory Exception when Truncating Log #97

Closed bchavez closed 8 years ago

bchavez commented 8 years ago

_Do not do this in the Log_: https://github.com/bchavez/RethinkDb.Driver/blob/master/Source/RethinkDb.Driver/Log.cs#L82

The Log's recursive Filter(msg) will blow the callstack (resulting in OutOfMemory exception) when the protocol trace contains hundreds of binary terms that the filter needs to truncate. WTF was I smoking???

:floppydisk: :walking: [**"Every move you carefully choose and... A shame you keep on losing..."**_](https://www.youtube.com/watch?v=yy9HAA5ODJA)