It is usually a very bad idea. Among other reasons, in many languages, db.foo.save(db.foo.findOne()) might insert a new object rather than updating the existing one.

We will need some way to disable the warnings for the few cases where it is necessary, such as map-reduce output.

Maybe this is best handled by warning in the driver rather than the server.