He's not going to remain a general is he? If that's the case, I think its very inappropriate. However, I don't feel that we should rule people out for the position based on their current positions.
The question should be 'is he qualified? and how will his background affect the work he does?' I wouldn't totally rule out those in the military, they could be the best choice for the job.