> On March 20, 2015, 1:53 a.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/client/cli/update.py, line 248 > > <https://reviews.apache.org/r/32276/diff/1/?file=901054#file901054line248> > > > > You can use frozenset here as well. It's a shame thrift sets are > > mutable by default (which causes them to be unhashable).
Good call, done. - Bill ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32276/#review77153 ----------------------------------------------------------- On March 20, 2015, 1:53 a.m., Bill Farner wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/32276/ > ----------------------------------------------------------- > > (Updated March 20, 2015, 1:53 a.m.) > > > Review request for Aurora and Kevin Sweeney. > > > Repository: aurora > > > Description > ------- > > Fixes this error: > ``` > $ aurora update list devcluster/vagrant/test/http_example --status active > Traceback (most recent call last): > File "/usr/local/bin/aurora/.bootstrap/_pex/pex.py", line 272, in execute > self.execute_entry(entry_point, args) > File "/usr/local/bin/aurora/.bootstrap/_pex/pex.py", line 320, in > execute_entry > runner(entry_point) > File "/usr/local/bin/aurora/.bootstrap/_pex/pex.py", line 343, in > execute_pkg_resources > runner() > File "/usr/local/bin/aurora/apache/aurora/client/cli/client.py", line 95, > in proxy_main > File "/usr/local/bin/aurora/apache/aurora/client/cli/__init__.py", line > 329, in execute > File "/usr/local/bin/aurora/apache/aurora/client/cli/__init__.py", line > 306, in _execute > File "/usr/local/bin/aurora/apache/aurora/client/cli/__init__.py", line > 382, in execute > File "apache/aurora/client/cli/update.py", line 315, in execute > TypeError: unhashable type: 'set' > ``` > > > Diffs > ----- > > src/main/python/apache/aurora/client/cli/update.py > 2168e99a315dd2916086100589c8345cd3a2c4ff > > Diff: https://reviews.apache.org/r/32276/diff/ > > > Testing > ------- > > > Thanks, > > Bill Farner > >
