Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

> On June 4, 2015, 4:55 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/config/thrift.py, line 218 > > > > > > Why set this at all? Does the scheduler read this field? > > Brian Wickman wrote: > it is

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

> On June 4, 2015, 11:55 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/config/thrift.py, line 218 > > > > > > Why set this at all? Does the scheduler read this field? it is not. i will leave it up to A

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/#review88188 --- Ship it! Ship It! - Joe Smith On June 1, 2015, 11:05 a.m., Brian

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/#review86742 --- Ship it! src/main/python/apache/aurora/config/thrift.py

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/#review86028 --- Ship it! Master (827b9ab) is green with this patch. ./build-suppo

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/ --- (Updated June 1, 2015, 6:05 p.m.) Review request for Aurora and Kevin Sweeney.

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/#review85258 --- Master (6db13ba) is red with this patch. ./build-support/jenkins/b

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/ --- (Updated May 26, 2015, 9:03 p.m.) Review request for Aurora and Kevin Sweeney.

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

> On May 16, 2015, 1:17 a.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/config/schema/base.py, line 98 > > > > > > Would be nice to have a wrapper here like Deprecated(Map(String, > > String)) You filed h

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

> On May 15, 2015, 6:17 p.m., Kevin Sweeney wrote: > > src/main/python/apache/aurora/config/__init__.py, lines 255-257 > > > > > > Delete this property entirely rather than return a dummy value? > > Presumably anythin

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/#review84021 --- Ship it! Master (07ab0bb) is green with this patch. ./build-suppo

Re: Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/#review84010 --- src/main/python/apache/aurora/config/__init__.py

Review Request 34300: Do better sanitation on the client side when encountering unbound pystachio refs

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/34300/ --- Review request for Aurora and Kevin Sweeney. Repository: aurora Description -