{"action":"created","issue":{"url":"https://api.github.com/repos/pietermartin/sqlg/issues/125","repository_url":"https://api.github.com/repos/pietermartin/sqlg","labels_url":"https://api.github.com/repos/pietermartin/sqlg/issues/125/labels{/name}","comments_url":"https://api.github.com/repos/pietermartin/sqlg/issues/125/comments","events_url":"https://api.github.com/repos/pietermartin/sqlg/issues/125/events","html_url":"https://github.com/pietermartin/sqlg/issues/125","id":195875060,"number":125,"title":"Lazy traversal and query execution clashes","user":{"login":"pietermartin","id":439356,"avatar_url":"https://avatars.githubusercontent.com/u/439356?v=3","gravatar_id":"","url":"https://api.github.com/users/pietermartin","html_url":"https://github.com/pietermartin","followers_url":"https://api.github.com/users/pietermartin/followers","following_url":"https://api.github.com/users/pietermartin/following{/other_user}","gists_url":"https://api.github.com/users/pietermartin/gists{/gist_id}","starred_url":"https://api.github.com/users/pietermartin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/pietermartin/subscriptions","organizations_url":"https://api.github.com/users/pietermartin/orgs","repos_url":"https://api.github.com/users/pietermartin/repos","events_url":"https://api.github.com/users/pietermartin/events{/privacy}","received_events_url":"https://api.github.com/users/pietermartin/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":2,"created_at":"2016-12-15T18:07:16Z","updated_at":"2016-12-17T13:55:34Z","closed_at":null,"body":"Sqlg executes many queries per traversal. It executes the queries lazily.\r\nIf while looping the traversal elements are inserted or modified then a subsequent query might return the inserted or modified element.\r\nThis breaks the traversal scope semantics.\r\n\r\n`TestVertexEdges.testBothEOnEdgeToSelf ` reveals this error.\r\nI have for now hacked `SqlgVertex.edges` to eagerly iterate the traversal but its not good enough.\r\n\r\nEither loose laziness or execute all queries at once and then lazily iterate the results."},"comment":{"url":"https://api.github.com/repos/pietermartin/sqlg/issues/comments/267764193","html_url":"https://github.com/pietermartin/sqlg/issues/125#issuecomment-267764193","issue_url":"https://api.github.com/repos/pietermartin/sqlg/issues/125","id":267764193,"user":{"login":"JPMoresmau","id":122273,"avatar_url":"https://avatars.githubusercontent.com/u/122273?v=3","gravatar_id":"","url":"https://api.github.com/users/JPMoresmau","html_url":"https://github.com/JPMoresmau","followers_url":"https://api.github.com/users/JPMoresmau/followers","following_url":"https://api.github.com/users/JPMoresmau/following{/other_user}","gists_url":"https://api.github.com/users/JPMoresmau/gists{/gist_id}","starred_url":"https://api.github.com/users/JPMoresmau/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/JPMoresmau/subscriptions","organizations_url":"https://api.github.com/users/JPMoresmau/orgs","repos_url":"https://api.github.com/users/JPMoresmau/repos","events_url":"https://api.github.com/users/JPMoresmau/events{/privacy}","received_events_url":"https://api.github.com/users/JPMoresmau/received_events","type":"User","site_admin":false},"created_at":"2016-12-17T13:55:34Z","updated_at":"2016-12-17T13:55:34Z","body":"FYI just to confirm the two tests in TestVertexEdges pass with TinkerGraph-Gremlin, so you're right about the current behavior breaking the semantics."}}
{"actor":{"display_login":"JPMoresmau"}}