{"action":"created","issue":{"url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1090","repository_url":"https://api.github.com/repos/FasterXML/jackson-databind","labels_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1090/labels{/name}","comments_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1090/comments","events_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1090/events","html_url":"https://github.com/FasterXML/jackson-databind/issues/1090","id":127852712,"number":1090,"title":"Merge functionality of `jackson-module-parameter-names`, dynamically","user":{"login":"cowtowncoder","id":55065,"avatar_url":"https://avatars.githubusercontent.com/u/55065?v=3","gravatar_id":"","url":"https://api.github.com/users/cowtowncoder","html_url":"https://github.com/cowtowncoder","followers_url":"https://api.github.com/users/cowtowncoder/followers","following_url":"https://api.github.com/users/cowtowncoder/following{/other_user}","gists_url":"https://api.github.com/users/cowtowncoder/gists{/gist_id}","starred_url":"https://api.github.com/users/cowtowncoder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cowtowncoder/subscriptions","organizations_url":"https://api.github.com/users/cowtowncoder/orgs","repos_url":"https://api.github.com/users/cowtowncoder/repos","events_url":"https://api.github.com/users/cowtowncoder/events{/privacy}","received_events_url":"https://api.github.com/users/cowtowncoder/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":13,"created_at":"2016-01-21T06:04:44Z","updated_at":"2016-09-20T03:20:47Z","closed_at":"2016-05-20T19:19:57Z","body":"It should be possible to start to add Java 8 support in dynamically in Jackson 2.8, but without requiring use of Java 8 -- if running on JDK 7, things should degrade gracefully.\r\n\r\nThe simplest piece of functionality to merge in should be that of `jackson-module-parameter-names` (see ../../../jackson-module-parameter-names), as it just adds discovery of parameter names.\r\n\r\nAnother thing that would go well with this would be a new `MapperFeature`, to allow changing of default handling of 1-parameter creators -- default of \"delegating\" could be overridable. For this, a separate issue should be filed, if and when implementing this RFE."},"comment":{"url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/comments/248191055","html_url":"https://github.com/FasterXML/jackson-databind/issues/1090#issuecomment-248191055","issue_url":"https://api.github.com/repos/FasterXML/jackson-databind/issues/1090","id":248191055,"user":{"login":"cowtowncoder","id":55065,"avatar_url":"https://avatars.githubusercontent.com/u/55065?v=3","gravatar_id":"","url":"https://api.github.com/users/cowtowncoder","html_url":"https://github.com/cowtowncoder","followers_url":"https://api.github.com/users/cowtowncoder/followers","following_url":"https://api.github.com/users/cowtowncoder/following{/other_user}","gists_url":"https://api.github.com/users/cowtowncoder/gists{/gist_id}","starred_url":"https://api.github.com/users/cowtowncoder/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cowtowncoder/subscriptions","organizations_url":"https://api.github.com/users/cowtowncoder/orgs","repos_url":"https://api.github.com/users/cowtowncoder/repos","events_url":"https://api.github.com/users/cowtowncoder/events{/privacy}","received_events_url":"https://api.github.com/users/cowtowncoder/received_events","type":"User","site_admin":false},"created_at":"2016-09-20T03:20:47Z","updated_at":"2016-09-20T03:20:47Z","body":"Actually maybe there is something to just adding a simple dependency and registration: but with a minor twist: instead of trying to do it all from `jackson-databind`, `ObjectMapper` -- which has the real problem of cyclic dependency -- could do following:\r\n\r\n1. Change `jackson-databind` to multi-module project\r\n2. Add second sub-project like `jackson-java8-databind`, which has dependency to both `jackson-databind` and java8 modules (actually, possibly it could and should directly add support for other java 8 features, `Optional` types, parameter names), and also adds `Java8ObjectMapper` with auto-registration\r\n\r\nIt could also be possible to use shading for (2) to reduce number of pieces to base `jackson-databind` and then all java8 features, but that's mostly just icing on the cake.\r\n\r\nHow would this sound? It would still allow use of \"classic\" `ObjectMapper`. For non-json formats it would either:\r\n\r\n1. Expose no change, for simple formats that do not sub-class `ObjectMapper` (cbor, smile at least)\r\n2. Be slightly cumbersome for other formats that do have optional or mandatory sub-type (mandatory for XML, optional for CSV, protobuf etc).\r\n\r\nSome other considerations:\r\n\r\n- It might not be immediately possible to deprecate java 8 modules, esp. if old ObjectMapper is still needed for some dataformats\r\n\r\n"}}
{"actor":{"display_login":"cowtowncoder"}}