{"action":"created","issue":{"url":"https://api.github.com/repos/rstats-db/DBI/issues/83","repository_url":"https://api.github.com/repos/rstats-db/DBI","labels_url":"https://api.github.com/repos/rstats-db/DBI/issues/83/labels{/name}","comments_url":"https://api.github.com/repos/rstats-db/DBI/issues/83/comments","events_url":"https://api.github.com/repos/rstats-db/DBI/issues/83/events","html_url":"https://github.com/rstats-db/DBI/pull/83","id":133694531,"number":83,"title":"R implementation of sqlParseVariablesImpl","user":{"login":"hannesmuehleisen","id":227792,"avatar_url":"https://avatars.githubusercontent.com/u/227792?v=3","gravatar_id":"","url":"https://api.github.com/users/hannesmuehleisen","html_url":"https://github.com/hannesmuehleisen","followers_url":"https://api.github.com/users/hannesmuehleisen/followers","following_url":"https://api.github.com/users/hannesmuehleisen/following{/other_user}","gists_url":"https://api.github.com/users/hannesmuehleisen/gists{/gist_id}","starred_url":"https://api.github.com/users/hannesmuehleisen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hannesmuehleisen/subscriptions","organizations_url":"https://api.github.com/users/hannesmuehleisen/orgs","repos_url":"https://api.github.com/users/hannesmuehleisen/repos","events_url":"https://api.github.com/users/hannesmuehleisen/events{/privacy}","received_events_url":"https://api.github.com/users/hannesmuehleisen/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":27,"created_at":"2016-02-15T11:52:55Z","updated_at":"2016-03-02T14:30:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/rstats-db/DBI/pulls/83","html_url":"https://github.com/rstats-db/DBI/pull/83","diff_url":"https://github.com/rstats-db/DBI/pull/83.diff","patch_url":"https://github.com/rstats-db/DBI/pull/83.patch"},"body":"Fixes #82."},"comment":{"url":"https://api.github.com/repos/rstats-db/DBI/issues/comments/191260249","html_url":"https://github.com/rstats-db/DBI/pull/83#issuecomment-191260249","issue_url":"https://api.github.com/repos/rstats-db/DBI/issues/83","id":191260249,"user":{"login":"hadley","id":4196,"avatar_url":"https://avatars.githubusercontent.com/u/4196?v=3","gravatar_id":"","url":"https://api.github.com/users/hadley","html_url":"https://github.com/hadley","followers_url":"https://api.github.com/users/hadley/followers","following_url":"https://api.github.com/users/hadley/following{/other_user}","gists_url":"https://api.github.com/users/hadley/gists{/gist_id}","starred_url":"https://api.github.com/users/hadley/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hadley/subscriptions","organizations_url":"https://api.github.com/users/hadley/orgs","repos_url":"https://api.github.com/users/hadley/repos","events_url":"https://api.github.com/users/hadley/events{/privacy}","received_events_url":"https://api.github.com/users/hadley/received_events","type":"User","site_admin":false},"created_at":"2016-03-02T14:30:36Z","updated_at":"2016-03-02T14:30:36Z","body":"My main concern the C++ used a state machine which is easy to reason about, and easy to adapt to changing requirements. I can't easily identify the strategy used in the new code, which means that if requirements change, @hannesmuehleisen will have to make the changes.   That said, I don't think this code is likely to change much in the future, so it's not a huge concern."}}