[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-10 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/199 @isper3at thanks! Merging now... --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-10 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/199 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/382/ --- If your project

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-10 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/199 @pujav65 totally agree. That's why I added the "please get some metrics" portion. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-10 Thread pujav65
Github user pujav65 commented on the issue: https://github.com/apache/incubator-rya/pull/199 The reason I wanted it broken apart was so this could be merged and more time could be spent figuring out if hashing is necessary. So I say merge this and do that on a follow on or. --- If

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-09 Thread pujav65
Github user pujav65 commented on the issue: https://github.com/apache/incubator-rya/pull/199 looks good to me. aaron's different hashing suggestion can be done later -- add it to jira to track if you don't want to do it now. --- If your project is set up for it, you can reply to

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-08 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/199 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/372/ --- If your project

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-08 Thread isper3at
Github user isper3at commented on the issue: https://github.com/apache/incubator-rya/pull/199 asfbot build --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so,

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-08 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/199 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/371/Failed Tests:

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-08 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/199 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/368/Failed Tests:

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-08 Thread asfgit
Github user asfgit commented on the issue: https://github.com/apache/incubator-rya/pull/199 Refer to this link for build results (access rights to CI server needed): https://builds.apache.org/job/incubator-rya-master-with-optionals-pull-requests/369/Failed Tests:

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-07 Thread isper3at
Github user isper3at commented on the issue: https://github.com/apache/incubator-rya/pull/199 sure, shouldn't be an issue --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled

[GitHub] incubator-rya issue #199: RYA-316 Long OBJ string

2017-08-07 Thread amihalik
Github user amihalik commented on the issue: https://github.com/apache/incubator-rya/pull/199 @isper3at Since this was so straightforward, can you go ahead and hash the S and O as well? --- If your project is set up for it, you can reply to this email and have your reply appear on