Why includes jackson-datatype-jsr310 in terasoluna-gfw-common-dependencies?

[kazuki43zoo]

Description

In current pom definition, the terasoluna-gfw-common-dependencies has dependency for the jackson-datatype-jsr310 as follow:

terasoluna-gfw/terasoluna-gfw-dependencies/terasoluna-gfw-common-dependencies/pom.xml

Lines 28 to 31 in 01606ff

	<dependency>
	<groupId>com.fasterxml.jackson.datatype</groupId>
	<artifactId>jackson-datatype-jsr310</artifactId>
	</dependency>

Is correct this dependency definition?
I think better this dependency separate like the terasoluna-gfw-jodatime-dependencies.
What do you think?

Possible Solutions

(Please write ideas or candidates of solutions for the problem if you have)

Affects Version/s

• 5.X.X.RELEASE
• 1.X.X.RELEASE

Fix Version/s

(To be written later by project member)

• 5.4.0 (master)
• 5.3.1 (5.3.x)

Final Solution

(To be written later by project member)

Migrating Guide

(To be selected later by project member)

• Required
• Not Required

Issue Links

• #XXX

[itouyun]

Thank you for your making issue!
Would we consider with #736 ?

@yoshikawaa
Please investigate this issue definition of jackson-datatype-jsr310.

[yoshikawaa]

@itouyun @kazuki43zoo
jackson-datatype-jsr310 was added at #562 (Support JSR310 on JSON and MyBatis).
But I could not find a certain reason to add it to terasoluna-gfw-common-dependencies.

I think it is correct to create a library like terasoluna-gfw-jsr310-dependencies or terasoluna-gfw-datetime-dependencies and move it to that. (I agree opinion of @kazuki43zoo )