{"action":"created","issue":{"url":"https://api.github.com/repos/apache/jena/issues/189","repository_url":"https://api.github.com/repos/apache/jena","labels_url":"https://api.github.com/repos/apache/jena/issues/189/labels{/name}","comments_url":"https://api.github.com/repos/apache/jena/issues/189/comments","events_url":"https://api.github.com/repos/apache/jena/issues/189/events","html_url":"https://github.com/apache/jena/pull/189","id":187606904,"number":189,"title":"JENA-1258 NodeFmtLib calls JenaSystem initializer","user":{"login":"stain","id":253413,"avatar_url":"https://avatars.githubusercontent.com/u/253413?v=3","gravatar_id":"","url":"https://api.github.com/users/stain","html_url":"https://github.com/stain","followers_url":"https://api.github.com/users/stain/followers","following_url":"https://api.github.com/users/stain/following{/other_user}","gists_url":"https://api.github.com/users/stain/gists{/gist_id}","starred_url":"https://api.github.com/users/stain/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stain/subscriptions","organizations_url":"https://api.github.com/users/stain/orgs","repos_url":"https://api.github.com/users/stain/repos","events_url":"https://api.github.com/users/stain/events{/privacy}","received_events_url":"https://api.github.com/users/stain/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2016-11-07T01:28:13Z","updated_at":"2016-11-08T12:31:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/apache/jena/pulls/189","html_url":"https://github.com/apache/jena/pull/189","diff_url":"https://github.com/apache/jena/pull/189.diff","patch_url":"https://github.com/apache/jena/pull/189.patch"},"body":"Fixes [JENA-1258](https://issues.apache.org/jira/browse/JENA-1258) in `NodeFmtLib`\r\n\r\nNote that the added `TestNodeFmtInit` test case would not normally find the bug ran from Maven as earlier tests would have caused initialization, but it can be tested from say Eclipse."},"comment":{"url":"https://api.github.com/repos/apache/jena/issues/comments/259124635","html_url":"https://github.com/apache/jena/pull/189#issuecomment-259124635","issue_url":"https://api.github.com/repos/apache/jena/issues/189","id":259124635,"user":{"login":"afs","id":101362,"avatar_url":"https://avatars.githubusercontent.com/u/101362?v=3","gravatar_id":"","url":"https://api.github.com/users/afs","html_url":"https://github.com/afs","followers_url":"https://api.github.com/users/afs/followers","following_url":"https://api.github.com/users/afs/following{/other_user}","gists_url":"https://api.github.com/users/afs/gists{/gist_id}","starred_url":"https://api.github.com/users/afs/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/afs/subscriptions","organizations_url":"https://api.github.com/users/afs/orgs","repos_url":"https://api.github.com/users/afs/repos","events_url":"https://api.github.com/users/afs/events{/privacy}","received_events_url":"https://api.github.com/users/afs/received_events","type":"User","site_admin":false},"created_at":"2016-11-08T12:31:50Z","updated_at":"2016-11-08T12:31:50Z","body":"The better fix is to initialize in `NodeFactory` (and `Node` for legacy reasons) only. I have tested this in a build.\r\n\r\nThere is little point in a specific test because tests run in the overall suite; it's almost confusing to have a test that doesn't. With initialization in `NodeFactory` this will get tested anyway!\r\n\r\nAlso, it would help to unroll ARQConstants (for RDF, RDFS, OWL, XSD) so it does not need JenaSystem.init and then it is, with care, init-safe.\r\n\r\nSee PR#192 for that all made real.\r\n\r\nThe original example works when run separately to ensure minimal system initialization."}}
{"actor":{"display_login":"afs"}}