Some example code to answer the question of when to move results into a let.

what-to-move-to-let.clj

;; using a let to store intermediat results
(defn recommendations [scores user]
  (let [user-row (.viewRow (:matrix scores)
                           (get (:row-mapping scores)
                                user))
        sorted-by-score (sort-by #(get % 1)
                                 (non-zero user-row))]
    (map (fn [[item, score]]
           {:item item, :score score})
         sorted-by-score)))


;; don't use any let
(defn recommendations [scores user]
  (map (fn [[item, score]]
           {:item item, :score score})
       (sort-by #(get % 1)
                (non-zero (.viewRow (:matrix scores)
                                    (get (:row-mapping scores)
                                         user))))))

I like it. :)
I still need to get used to the ->> and -> operators.
Often times I find that wen I need to use use directly I end up needing the other one.

For now I can only see them as a refactoring.

(defn recommendations [scores user]
  (->> user
       (get (:row-mapping scores))
       (.viewRow (:matrix scores)) ;; User row
       non-zero
       (sort-by #(get % 1)) ;; Sorted by score
       (map (fn [[item, score]]
              {:item item, :score score}))))

Thanks for the comment. I will refactor my code, because this version is very nice.