Esri / spatial-framework-for-hadoop

The Spatial Framework for Hadoop allows developers and data scientists to use the Hadoop data processing system for spatial data analysis.
Apache License 2.0
367 stars 159 forks source link

Fix wrong String comparison in EnclosedJsonRecordReader #88

Closed komamitsu closed 9 years ago

komamitsu commented 9 years ago

parser.getCurrentName() should be compared using String#equals.

randallwhitman commented 9 years ago

By inspection/review, this patch looks correct.

climbage commented 9 years ago

@randallwhitman Looks good to me - merge?

randallwhitman commented 9 years ago

Passes code review. Do we have any tests to run before merge? @climbage @smambrose

randallwhitman commented 9 years ago

Thanks @komamitsu !