Nope, I meant R8.Did you mean R7? It was a demo value, since the power supply wasn't defined.
My point is that using ~3k for R8 as a current source, the output will have more drive for the load.Given that R8 turns off the transistor almost anything will work. The current for the transistor comes directly from the output of the LM339, and if you are providing 30 ma to the LED the LM339 is sinking much, much less (0.3ma?). This is one of my better designs.