{"action":"created","issue":{"url":"https://api.github.com/repos/hornetq/hornetq/issues/1988","labels_url":"https://api.github.com/repos/hornetq/hornetq/issues/1988/labels{/name}","comments_url":"https://api.github.com/repos/hornetq/hornetq/issues/1988/comments","events_url":"https://api.github.com/repos/hornetq/hornetq/issues/1988/events","html_url":"https://github.com/hornetq/hornetq/pull/1988","id":57297467,"number":1988,"title":"Management Methods for first message timestamp","user":{"login":"dpocock","id":1236687,"avatar_url":"https://avatars.githubusercontent.com/u/1236687?v=3","gravatar_id":"","url":"https://api.github.com/users/dpocock","html_url":"https://github.com/dpocock","followers_url":"https://api.github.com/users/dpocock/followers","following_url":"https://api.github.com/users/dpocock/following{/other_user}","gists_url":"https://api.github.com/users/dpocock/gists{/gist_id}","starred_url":"https://api.github.com/users/dpocock/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dpocock/subscriptions","organizations_url":"https://api.github.com/users/dpocock/orgs","repos_url":"https://api.github.com/users/dpocock/repos","events_url":"https://api.github.com/users/dpocock/events{/privacy}","received_events_url":"https://api.github.com/users/dpocock/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-02-11T10:16:02Z","updated_at":"2015-02-20T22:02:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/hornetq/hornetq/pulls/1988","html_url":"https://github.com/hornetq/hornetq/pull/1988","diff_url":"https://github.com/hornetq/hornetq/pull/1988.diff","patch_url":"https://github.com/hornetq/hornetq/pull/1988.patch"},"body":"This basically extends the getFirstMessageAsJSON() concept to extract just the timestamp.\r\n\r\nFor large messages in particular, this is much more efficient than returning the whole message as JSON and then extracting the timestamp.\r\n\r\nI tested with JConsole and it is OK, but there are a few things that could be done differently, if anybody has an opinion on these things I don't mind changing them:\r\n\r\n- should getFirstMessageTimestamp() return a java.util.Date object, the Java way of doing things, or return a long, just like the message header?\r\n\r\n- when there are no messages in the queue, should the methods return null?  There are two other possibilities that came to mind: if the return values are doubles, they could return NaN when the queue is empty.  The other possibility is throwing IllegalStateException.\r\n\r\nI'd like to have this merged into 2.3.x as well, so I've tried to keep it as simple as possible."},"comment":{"url":"https://api.github.com/repos/hornetq/hornetq/issues/comments/75327431","html_url":"https://github.com/hornetq/hornetq/pull/1988#issuecomment-75327431","issue_url":"https://api.github.com/repos/hornetq/hornetq/issues/1988","id":75327431,"user":{"login":"jbertram","id":1336948,"avatar_url":"https://avatars.githubusercontent.com/u/1336948?v=3","gravatar_id":"","url":"https://api.github.com/users/jbertram","html_url":"https://github.com/jbertram","followers_url":"https://api.github.com/users/jbertram/followers","following_url":"https://api.github.com/users/jbertram/following{/other_user}","gists_url":"https://api.github.com/users/jbertram/gists{/gist_id}","starred_url":"https://api.github.com/users/jbertram/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbertram/subscriptions","organizations_url":"https://api.github.com/users/jbertram/orgs","repos_url":"https://api.github.com/users/jbertram/repos","events_url":"https://api.github.com/users/jbertram/events{/privacy}","received_events_url":"https://api.github.com/users/jbertram/received_events","type":"User","site_admin":false},"created_at":"2015-02-20T22:02:04Z","updated_at":"2015-02-20T22:02:04Z","body":"These commits need to be squashed, no?"}}