{"action":"created","issue":{"url":"https://api.github.com/repos/traject-project/traject/issues/82","labels_url":"https://api.github.com/repos/traject-project/traject/issues/82/labels{/name}","comments_url":"https://api.github.com/repos/traject-project/traject/issues/82/comments","events_url":"https://api.github.com/repos/traject-project/traject/issues/82/events","html_url":"https://github.com/traject-project/traject/issues/82","id":57084330,"number":82,"title":"Make Marc4JReader default on jruby somehow?","user":{"login":"jrochkind","id":149304,"avatar_url":"https://avatars.githubusercontent.com/u/149304?v=3","gravatar_id":"","url":"https://api.github.com/users/jrochkind","html_url":"https://github.com/jrochkind","followers_url":"https://api.github.com/users/jrochkind/followers","following_url":"https://api.github.com/users/jrochkind/following{/other_user}","gists_url":"https://api.github.com/users/jrochkind/gists{/gist_id}","starred_url":"https://api.github.com/users/jrochkind/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jrochkind/subscriptions","organizations_url":"https://api.github.com/users/jrochkind/orgs","repos_url":"https://api.github.com/users/jrochkind/repos","events_url":"https://api.github.com/users/jrochkind/events{/privacy}","received_events_url":"https://api.github.com/users/jrochkind/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-02-09T20:23:20Z","updated_at":"2015-02-09T23:26:16Z","closed_at":null,"body":"@billdueber  finds that it's about 2x as fast at reading XML on Jruby than the MarcReader. (Not sure if that's MarcReader with nokogiri or not?). \r\n\r\nhttps://gist.github.com/billdueber/a388b2d0d3aec7f96134\r\n\r\nThis makes me less happy at it requiring extra setup to use -- for ruby newbies, installing/using gems can be non-trivial, and I want ruby newbies to get high performance out of the box. \r\n\r\nPossible ideas:\r\n\r\n* Make traject default to Marc4JReader instead of MarcReader under jruby, and, either one or two of:\r\n    * 1) Make a jruby version of the gem, which is _identical_ *but for* including Marc4JReader in it's gemspec. \r\n    * 2) Make all versions of traject depend on Marc4JReader in gemspec, but it simply won't be used unless under JRuby (and would raise if you tried).  That would require making Marc4JReader willing to be loaded on MRI -- but doing anything other than loading it would of course raise. \r\n\r\n* Simply warn if they are on JRuby and using MarcReader, log that they should use Marc4JReader instead, but leave them to figure out how to get it set up themselves. (Last resort from my point of view). \r\n\r\n"},"comment":{"url":"https://api.github.com/repos/traject-project/traject/issues/comments/73613478","html_url":"https://github.com/traject-project/traject/issues/82#issuecomment-73613478","issue_url":"https://api.github.com/repos/traject-project/traject/issues/82","id":73613478,"user":{"login":"billdueber","id":114006,"avatar_url":"https://avatars.githubusercontent.com/u/114006?v=3","gravatar_id":"","url":"https://api.github.com/users/billdueber","html_url":"https://github.com/billdueber","followers_url":"https://api.github.com/users/billdueber/followers","following_url":"https://api.github.com/users/billdueber/following{/other_user}","gists_url":"https://api.github.com/users/billdueber/gists{/gist_id}","starred_url":"https://api.github.com/users/billdueber/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/billdueber/subscriptions","organizations_url":"https://api.github.com/users/billdueber/orgs","repos_url":"https://api.github.com/users/billdueber/repos","events_url":"https://api.github.com/users/billdueber/events{/privacy}","received_events_url":"https://api.github.com/users/billdueber/received_events","type":"User","site_admin":false},"created_at":"2015-02-09T23:26:16Z","updated_at":"2015-02-09T23:26:16Z","body":"What would really rankle about removing the 'java' platform is that it implicitly makes a claim that it'll work under any ruby, and it won't.  You're right -- I would prefer the second option. \r\n\r\nCan a gemspec just have a `if spec.platform =~ /java/`... in it? Or do we really need to have multiple gemspecs?"}}