{"comment":{"url":"https://api.github.com/repos/hchiorean/modeshape/comments/14687248","html_url":"https://github.com/hchiorean/modeshape/commit/2bb8520d8e80b79b5f498a33231e61c7bd3c33fa#commitcomment-14687248","id":14687248,"user":{"login":"hchiorean","id":995683,"avatar_url":"https://avatars.githubusercontent.com/u/995683?v=3","gravatar_id":"","url":"https://api.github.com/users/hchiorean","html_url":"https://github.com/hchiorean","followers_url":"https://api.github.com/users/hchiorean/followers","following_url":"https://api.github.com/users/hchiorean/following{/other_user}","gists_url":"https://api.github.com/users/hchiorean/gists{/gist_id}","starred_url":"https://api.github.com/users/hchiorean/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hchiorean/subscriptions","organizations_url":"https://api.github.com/users/hchiorean/orgs","repos_url":"https://api.github.com/users/hchiorean/repos","events_url":"https://api.github.com/users/hchiorean/events{/privacy}","received_events_url":"https://api.github.com/users/hchiorean/received_events","type":"User","site_admin":false},"position":84,"line":84,"path":"modeshape-jcr/src/main/java/org/modeshape/jcr/txn/DefaultTransactionManagerLookup.java","commit_id":"2bb8520d8e80b79b5f498a33231e61c7bd3c33fa","created_at":"2015-11-30T17:01:56Z","updated_at":"2015-11-30T17:01:56Z","body":">Well, I was wondering if the code removing some of the Supplier<Optional<TransactionManager>> \r\n\r\nactually no, not without explicit casting which is pretty much the same thing. The `Stream.of` method takes a vararg `T` so the compiler complains (I think the reason is that `Object` is not a `FunctionalInterface` and therefore type erasure would not work with method references directly).\r\n\r\nThe last part of the comment - i.e. chaining with `orElse` make sense, so I'll make that change."}}