Agreed, on second thought it's weird to not do it here.

On Sep 5, 2013, at 5:34 PM, Chris Price <notifications@github.com> wrote:

> In src/com/puppetlabs/puppetdb/query/facts.clj:
> 
> > @@ -26,22 +25,23 @@
> >       ...
> >       {:certname <node> :name <fact> :value <value>}]"
> >    [node]
> > -  (-> (table :certname_facts)
> > -      (project [:certname :name :value])
> > -      (select (where (= :certname node)))
> > -      (deref)))
> > +  (sql/query-to-vec
> > +    ["SELECT certname, name, value FROM certname_facts WHERE certname = ?"
> > +     node]))
> >  
> >  (defn fact-names
> Just seems inconsistent not to, and it's pretty easy... so I didn't see a downside.
> 
> —
> Reply to this email directly or view it on GitHub.
>