Branch: refs/heads/data-api Home: https://github.com/jenkinsci/jenkins Commit: 62047330aab61e9d94d0ea582a3e26add06b5f6e https://github.com/jenkinsci/jenkins/commit/62047330aab61e9d94d0ea582a3e26add06b5f6e Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2018-09-28 (Fri, 28 Sep 2018)
Changed paths: M core/src/main/java/jenkins/data/DataModelFactory.java M core/src/main/java/jenkins/data/DefaultDataModelRegistry.java M core/src/main/java/jenkins/data/exportable/APIExportableModelFactory.java Log Message: ----------- Revert "DataModelFactory need a Type to find, not a Class" This reverts commit 05285432fdb4d772e6f8ae4984b4181c06d9cbfa. Commit: 36a82b4b089d63fcb94f59b89f2eecb022aabbe4 https://github.com/jenkinsci/jenkins/commit/36a82b4b089d63fcb94f59b89f2eecb022aabbe4 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2018-09-28 (Fri, 28 Sep 2018) Changed paths: M core/src/main/java/jenkins/data/DataContext.java M core/src/main/java/jenkins/data/DataModel.java M core/src/main/java/jenkins/data/DataModelRegistry.java M core/src/main/java/jenkins/data/DefaultDataModelFactoryImpl.java M core/src/main/java/jenkins/data/DefaultDataModelRegistry.java M core/src/main/java/jenkins/data/ReflectiveDataModelParameter.java M core/src/main/java/jenkins/data/Serializer.java Log Message: ----------- Nailing the consistency between Type vs Class DataModel is for Class, not for Type. I provided rationale in the javadoc of DataModel a bit. Commit: 8ab87a5387c269385c01e21b6a1e6bdedeb2a654 https://github.com/jenkinsci/jenkins/commit/8ab87a5387c269385c01e21b6a1e6bdedeb2a654 Author: Kohsuke Kawaguchi <k...@kohsuke.org> Date: 2018-09-28 (Fri, 28 Sep 2018) Changed paths: M core/src/main/java/jenkins/data/DataModel.java M core/src/main/java/jenkins/data/JsonSerializer.java M core/src/main/java/jenkins/data/ReflectiveDataModel.java M core/src/main/java/jenkins/data/Serializer.java M core/src/main/java/jenkins/data/exportable/APIExportableModelFactory.java M test/src/test/java/hudson/cli/SampleDataCLITest.java Log Message: ----------- DataModel is for big objects, so it should just deal with Mapping This squares well with the type annotation to deal with polymorphism Compare: https://github.com/jenkinsci/jenkins/compare/747f36027798...8ab87a5387c2 **NOTE:** This service has been marked for deprecation: https://developer.github.com/changes/2018-04-25-github-services-deprecation/ Functionality will be removed from GitHub.com on January 31st, 2019. -- You received this message because you are subscribed to the Google Groups "Jenkins Commits" group. To unsubscribe from this group and stop receiving emails from it, send an email to jenkinsci-commits+unsubscr...@googlegroups.com. For more options, visit https://groups.google.com/d/optout.