Details
-
Improvement
-
Status: Resolved
-
Minor
-
Resolution: Won't Fix
-
1.99.5
-
None
-
None
Description
SQOOP-1995 fixed a NPE, which is adding a schema NPE check in 6 get/set FooData methods. vybs explained that the IDF class can be instantiated by reflection, which will call the default no arg constructor, so that the property schema will not be assigned expectedly.
IMO, although the patch can guard invalid access to a null schema, but as the getter and setter methods have nothing to do with schema and there is no comment left, it will be hard to understand why we have such code after couple of months.
It seems a IDF class should have a not null schema the whole life time. I've some options:
1. We check null schema where a NPE happens
2. Disallow reflection of IDF class or search all code occurrences to make sure schema will be set properly (I think we already have test cases to guard use cases)
3. Use some design by contract framework (https://github.com/nhatminhle/cofoja) to define invariant, e.g. @Invariant(schema != null).
Attachments
Attachments
Issue Links
- links to
