What is the 330K at the output supposed to do? Oh, that is a LED?
A high brightness LED will illuminate brightly enough to be seen even in sunlight with 5mA flowing through it; 10 is better.
With 9V out of the battery, and two VceSat drops (~0.3V ea), less the Forward voltage drop across the LED (~1.8 to 3.3V, depending on color), the LED current limiting resistor will have ~5.5V across it, mean that to get 5mA, it would have to be 5.5/0.005 = 1K.
What is the 10K from emitter to ground supposed to do?