Clojurians Log v2

https://clojurians.slack.com/archives/C06MAR553/p1648830782525509

this might be helpful for optimizing the io

If you're interested in speeding things up, I highly recommend clj-async-profiler to find the slow parts: • https://github.com/clojure-goes-fast/clj-async-profiler • http://clojure-goes-fast.com/blog/profiling-tool-async-profiler/

I'll play around with this now. Part of the exercise for me was to check out the benchmarking/profiling tools. I would still like to see how others approach this as I'm sure there is so much low hanging fruit for performance gains. I'm looking for a guide on as opposed to slurp ing a 43 mb file (unless that is efficient?)

I have a transducer version that uses line-seq , but performance wasn't a goal, so I don't claim it's efficient. It might be a starting point for comparison though.

(def line-counts
    (with-open [is (io/input-stream fname)]
      (let [is (if (clojure.string/ends-with? fname ".gz")
                 (java.util.zip.GZIPInputStream. is)
                 is)
            rdr (io/reader is)
            ret (into
                 []
                 (comp (map count))
                 (line-seq rdr))]
        ret)))

If I was interested in performance, then I would definitely run it with a profiler and look at the flame graph to see where most of the time is being spent.

Ok, cool, I'll try and wrap my head around this. I'm getting permission denied errors when trying to run some required config commands on that profiler you sent me. Trying to work around that now.

I think my issue was trying to use this line-seq rdr part but continuously tallying the frequencies as it was processed. I couldn't get that worked out.

I guess since you're splitting on words, I might start with java.util.Scanner rather than line-seq

Nice. That seems to be what his Go solution used as well. I consider lack of knowledge of the java ecosystem one of my biggest clojure holes and it seems IO is full of it.

yea, I'm trying to give pointers without just writing a solution, but there is some amount of just java arcana

It's also been long enough since I've learned java where I'm not totally sure if Scanner would be the recommended option if efficiency is a priority.

Yeah the first thing that popped up in a search mentioned it wasn't too efficient. haha. I'm thinking a "simple" idiomatic Clojure solution should at least be on par with these middle ground simple solutions presented in the article that take about 2.5-4 seconds.

here's at least a basic skeleton for using transducers with Scanner:

(import 'java.util.Scanner)

(defn scan [xform f init scanner]
  (let [f (xform f)]
    (loop [result init]
      (if (.hasNext ^Scanner scanner)
        (recur (f result (.next ^Scanner scanner)))
        result))))

(with-open [is (io/input-stream fname)]
  (scan (comp (map #(.length ^String %)))
        +
        0
        (Scanner. is)))

It currently just sums up the lengths of each word

to do word counts, you would need to come up with a different reducing function, f, an initial value, init, and potentially xform

Yeah, I'll play around with it. They have to all be lowercased too but not sure that will add much overhead

I appreciate the help!

yea, that's good thing to put in the xform

I won't be able to use that cool profiler btw. I'm on a pixelbook running a Linux VM but I guess the required kernal commands for the profiler aren't allowed for the host OS on my system.

you can try visuamvm, http://clojure-goes-fast.com/blog/profiling-tool-jvisualvm/

but I think it's really difficult to write efficient code without a profiler. It's like trying to dig a hole without a shovel.

makes sense to me. I got that profiler working so I'll play around with it.

Iirc the most relevant conclusion to this use case is not using a push back reader

@chase-lambert see anything interesting in the profiling?

If you want to work really hard you can so this with arrays and cut down dramatically on allocation

What does "push back" reader mean?

that's a java type that has an unread method

@smith.adriane still working on grokking visualvm. I would probably still need assistance on what it's actually showing me. This is why I'm ready to stop the solo self learning and try to get in as a junior developer or even just get involved in an open source project to be mentored by senior devs. I'm not progressing like I want by myself but I digress