> On Aug. 26, 2014, 6:55 p.m., Abraham Elmahrek wrote: > > common/src/test/java/org/apache/sqoop/model/TestFormUtils.java, line 276 > > <https://reviews.apache.org/r/25074/diff/3/?file=669962#file669962line276> > > > > How about removing "name" here to have a variation that doesn't provide > > "name". Or maybe adding one more and a test case to validate it?
DForm supports this use case already. Hence I would keep this. - Veena ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25074/#review51626 ----------------------------------------------------------- On Aug. 26, 2014, 6:14 p.m., Veena Basavaraj wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/25074/ > ----------------------------------------------------------- > > (Updated Aug. 26, 2014, 6:14 p.m.) > > > Review request for Sqoop, Abraham Elmahrek, Gwen Shapira, and Jarek Cecho. > > > Bugs: SQOOP-1436 > https://issues.apache.org/jira/browse/SQOOP-1436 > > > Repository: sqoop-sqoop2 > > > Description > ------- > > https://issues.apache.org/jira/browse/SQOOP-1436 > As discussed in the SQOOP-1436, this feature gives the ability to specifiy a > custom form name. It is optional, if given it is persisted in the metadata > repo. Methods that convert the form fields to JSON have also been modified to > read the custom form name attribute. basic validation for uniqueness, length > and pattern have been added > > Existing unit tests modified to support the custom form name attribute > unit tests added for validation cases > > Note: there are formatting changes since I used the mandated eclipse > formatter.xml in the IDE > > > Diffs > ----- > > common/src/main/java/org/apache/sqoop/model/Form.java 4321582 > common/src/main/java/org/apache/sqoop/model/FormUtils.java 27db8af > common/src/main/java/org/apache/sqoop/model/ModelError.java 1f466fe > common/src/test/java/org/apache/sqoop/model/TestFormUtils.java 08dfa7b > > Diff: https://reviews.apache.org/r/25074/diff/ > > > Testing > ------- > > mvn test and integration tests pass. > > > Thanks, > > Veena Basavaraj > >
