[Java][FlightRPC] flight-core and flight-sql jars delivering same class names

[asfimport]

Hello. I am trying to uptake arrow flight sql. We have a check in out build to make sure that there are no overlapping class files in our project. When adding the flight sql dependency to our project the warning throws that flight-sql and flight-core overlap and the jars deliver the same class files.

Sample Warning: WARNING: CLASSPATH OVERLAP: These jars deliver the same class files: [flight-sql-7.0.0.jar, flight-core-7.0.0.jar] files: [org/apache/arrow/flight/impl/Flight$FlightDescriptor$1.class, org/apache/arrow/flight/impl/Flight$ActionOrBuilder.class

 
It seems that the classes generated by Flight.proto gets generated in both flight-sql and flight-core jars. Since these classes get generated in flight-core, and flight-sql is dependent on flight-core, can the generation of Flight.java and FlightServiceGrpc.java be removed from flight-sql and instead rely on it to be pulled directly from flight-core?
 
thanks in advance!

Reporter: Kevin Bambrick / @KBambrick1
Assignee: Kevin Bambrick / @KBambrick1

PRs and other links:

• GitHub Pull Request #13596

_{Note: This issue was originally created as ARROW-17055. Please see the migration documentation for further details.}

[asfimport]

David Li / @lidavidm:
Yup, those classes shouldn't make it into both jars. (What do you use for this check? Ideally we'd set up a similar one.)

CC @davisusanibar  

[asfimport]

David Li / @lidavidm:
Issue resolved by pull request 13596
#13596