Disable pretty printing ast nodes for now

We need and know how to fix excessive stack useage of perty; now's not the time for it.

If you want it back for debugging, just un-commment in ASTNode.
Speculative fix for Windows
Replace uses of Map.mapValues (which is lazy) with Eagerly.(mutable|immutable)mapValues

o Deprecate Materialized from commons (see comment there)
Merge pull request #3082 from boggle/eager-map-vals

Replace uses of Map.mapValues (which is lazy) with Eagerly.(mutable|immu...
Merge pull request #3080 from boggle/disable-perty

Disable pretty printing ast nodes for now
Initial identifer namespacing work
Make namespaceIdentifiers less invasive

The rewriter will only introduce special names for identifiers if
there's shadowing involved. Specifically, if an identifier is never
projected through from one scope to another, it will not be altered.
In addition, trivial projections such as `WITH n AS n` are also
understood not to introduce ambiguity to identifiers.
Simplify inliningContext given our new contracts for shadowing

This also allows for more aggresive inlining. See StatementConvertersTest
for examples of AST -> PlannerQuery conversions that are now more inlining
happy.
Improve the clustering of identifiers to address scoping limitations
Move the namespaceIdentifiers rewriter to Planner