Took a look at this one. I don't think we can make them final as we can only generate them after or in configureFieldMetadata. That being said, I can see the rationale for the longitude and latitude fields. As for the hash fields, I wonder if it's such a good idea as we probably will need a Map<Integer, String> to store the values. Waiting for your feedback before fixing this one. |