Comment 1 for bug 194031

Revision history for this message
James Henstridge (jamesh) wrote :

The main problem I see with your branch as it stands is the use of Variable.get().

If the variable is set to a lazy value your repr() implementations will emit resolve-lazy-value, which will run an SQL query to resolve the value and modify the variable's internal state. As the repr() method is mainly used in debugging, it would be good to avoid this.