Ryan Fu created CALCITE-3436:
--------------------------------

             Summary: CalciteConnectionConfigImpl.set obliterates previous 
property values
                 Key: CALCITE-3436
                 URL: https://issues.apache.org/jira/browse/CALCITE-3436
             Project: Calcite
          Issue Type: Bug
            Reporter: Ryan Fu


The cause is that `new Properties(properties)` doesn't copy the previous value 
as, say, `new HashMap(map)` does.

 
 * Add a unit test in PlannerTest, that if you call set(x, 1) then set(y, 2) on 
a CalciteConnectionConfigImpl, the result contains x and y. 
 * Consider potentially using `clone()` as Properties implements Cloneable



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to