{"action":"created","issue":{"url":"https://api.github.com/repos/apiman/apiman/issues/268","labels_url":"https://api.github.com/repos/apiman/apiman/issues/268/labels{/name}","comments_url":"https://api.github.com/repos/apiman/apiman/issues/268/comments","events_url":"https://api.github.com/repos/apiman/apiman/issues/268/events","html_url":"https://github.com/apiman/apiman/pull/268","id":116349984,"number":268,"title":"vertx with query params","user":{"login":"msavy","id":423513,"avatar_url":"https://avatars.githubusercontent.com/u/423513?v=3","gravatar_id":"","url":"https://api.github.com/users/msavy","html_url":"https://github.com/msavy","followers_url":"https://api.github.com/users/msavy/followers","following_url":"https://api.github.com/users/msavy/following{/other_user}","gists_url":"https://api.github.com/users/msavy/gists{/gist_id}","starred_url":"https://api.github.com/users/msavy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msavy/subscriptions","organizations_url":"https://api.github.com/users/msavy/orgs","repos_url":"https://api.github.com/users/msavy/repos","events_url":"https://api.github.com/users/msavy/events{/privacy}","received_events_url":"https://api.github.com/users/msavy/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-11-11T14:54:18Z","updated_at":"2015-11-11T15:04:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apiman/apiman/pulls/268","html_url":"https://github.com/apiman/apiman/pull/268","diff_url":"https://github.com/apiman/apiman/pull/268.diff","patch_url":"https://github.com/apiman/apiman/pull/268.patch"},"body":"So, my solution here allows things to done cleanly with PreparedStatement under the covers - however, as you rightly pointed out the `execute` method doesn't have parameters for some reason. I've worked around that by actually executing a `updateWithParams`.\r\n\r\nI'll make a PR upstream to see if they'll accept an addition of `executeWithParams`. Should be easy :). "},"comment":{"url":"https://api.github.com/repos/apiman/apiman/issues/comments/155806027","html_url":"https://github.com/apiman/apiman/pull/268#issuecomment-155806027","issue_url":"https://api.github.com/repos/apiman/apiman/issues/268","id":155806027,"user":{"login":"msavy","id":423513,"avatar_url":"https://avatars.githubusercontent.com/u/423513?v=3","gravatar_id":"","url":"https://api.github.com/users/msavy","html_url":"https://github.com/msavy","followers_url":"https://api.github.com/users/msavy/followers","following_url":"https://api.github.com/users/msavy/following{/other_user}","gists_url":"https://api.github.com/users/msavy/gists{/gist_id}","starred_url":"https://api.github.com/users/msavy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/msavy/subscriptions","organizations_url":"https://api.github.com/users/msavy/orgs","repos_url":"https://api.github.com/users/msavy/repos","events_url":"https://api.github.com/users/msavy/events{/privacy}","received_events_url":"https://api.github.com/users/msavy/received_events","type":"User","site_admin":false},"created_at":"2015-11-11T15:04:42Z","updated_at":"2015-11-11T15:04:42Z","body":"Roger\n\nOn Wed, 11 Nov 2015 at 15:03 Eric Wittmann <notifications@github.com> wrote:\n\n> This looks great! I didn't see the WithParams variants - but TBH I didn't\n> really look either. :)\n>\n> Before merging, can you update a couple of your unit tests to use params?\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/apiman/apiman/pull/268#issuecomment-155805807>.\n>\n"}}