ddf-project / DDF

Distributed DataFrame: Productivity = Power x Simplicity For Scientists & Engineers, on any Data Engine
http://ddf.io
Apache License 2.0
168 stars 42 forks source link

Modifying input param is a very bad idea #243

Closed lebinh closed 8 years ago

lebinh commented 8 years ago

This fix PE-1366. Root cause is that when the resolve method is called twice (the 1st time is to check if it a SQL source so that we can enforce authz, the 2nd time is to actual load the DDF) the 2nd time it will receive an incorrect options value. This fix clone the options and modify it instead.