Re: Review Request 53402: GEODE-2013: throw IllegalStateException if resource type is missing

2016-11-02 Thread Kirk Lund
> On Nov. 2, 2016, 8:26 p.m., Kevin Duling wrote: > > What is the possibility of an ArrayIndexOutOfBoundsException on that > > assignment? The ArrayIndexOutOfBoundsException message already contains the resourceTypeId because the array position is the type id. Still, it's a good idea and I'll

Re: Review Request 53402: GEODE-2013: throw IllegalStateException if resource type is missing

2016-11-02 Thread Darrel Schneider
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53402/#review154635 --- Ship it! Ship It! - Darrel Schneider On Nov. 2, 2016, 1:07

Re: Review Request 53402: GEODE-2013: throw IllegalStateException if resource type is missing

2016-11-02 Thread Jinmei Liao
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53402/#review154629 --- Ship it! Ship It! - Jinmei Liao On Nov. 2, 2016, 8:07 p.m.,

Review Request 53402: GEODE-2013: throw IllegalStateException if resource type is missing

2016-11-02 Thread Kirk Lund
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/53402/ --- Review request for geode, Anthony Baker, Darrel Schneider, Jinmei Liao, Jared