René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Refactor ExecuteQuery","articleBody":"If you look at the code of method [SqlPersistenceEngine::ExecuteQuery](https://github.com/NEventStore/NEventStore.Persistence.SQL/blob/accc7c627f4ad093e0dcd601043fc4eb511d95bd/src/NEventStore.Persistence.Sql/SqlPersistenceEngine.cs#L374) you can notice that nor connection nor transaction gets disposed if the code is successful. \n\nCompare ExecuteQuery with the implementation of [SqlPersistenEngine::ExecuteCommand](https://github.com/NEventStore/NEventStore.Persistence.SQL/blob/accc7c627f4ad093e0dcd601043fc4eb511d95bd/src/NEventStore.Persistence.Sql/SqlPersistenceEngine.cs#L443)  that correctly disposes everything.\n\nThis can lead to serious problem, and actually probably it did not generated error because the standard implementation of OpenTransaction returns null. \n\nI think that all the disposable object should be wrapped in using clause.\n","author":{"url":"https://github.com/alkampfergit","@type":"Person","name":"alkampfergit"},"datePublished":"2016-04-04T14:17:15.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/20/NEventStore.Persistence.SQL/issues/20"}