Skip to content

Commit d4f04c8

Browse files
rafael-tellesvfraga
authored andcommitted
Flight SQL Ratification Based On Community Feedback #7 (apache#98)
* Remove scope from 'hamcrest' dependency on java/pom.xml * Use flight top-level module on parent pom.xml instead of declaring each one * Avoid using getStatement inside StatementContext methods * Make StatementContext.getQuery() return String * Minor fixes on pom.xml * Move 'os-maven-plugin' to parent pom.xml * Update protobuf generation on pom.xml files * Use ClassLoader#getResource to get network.properties on TestFlightSql * Bind to any ephemeral port on TestFlightSql * Move JDBC-Arrow type default conversion from JdbcToArrowConfig to JdbcToArrowUtils * Micro-optimization: initialize ArrayList with the right size * Fix null-check on PreparedStatement#setParameters * Avoid wrapping vector into a ImmutableList and then into an ArrayList on FlightSqlExample#getTablesRoot * Remove null-check on VectorSchemaRoot on FlightSqlClient#setParameters() * Remove the need of separate cache for ResultSets * Add missing 'final' modifiers
1 parent c3fb8c7 commit d4f04c8

1 file changed

Lines changed: 32 additions & 0 deletions

File tree

java/pom.xml

Lines changed: 32 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -506,6 +506,38 @@
506506
</lifecycleMappingMetadata>
507507
</configuration>
508508
</plugin>
509+
510+
<plugin>
511+
<groupId>org.xolstice.maven.plugins</groupId>
512+
<artifactId>protobuf-maven-plugin</artifactId>
513+
<version>0.6.1</version>
514+
<configuration>
515+
<protocArtifact>com.google.protobuf:protoc:${dep.protobuf.version}:exe:${os.detected.classifier}</protocArtifact>
516+
<pluginId>grpc-java</pluginId>
517+
<pluginArtifact>io.grpc:protoc-gen-grpc-java:${dep.grpc.version}:exe:${os.detected.classifier}</pluginArtifact>
518+
</configuration>
519+
<executions>
520+
<execution>
521+
<id>proto-compile</id>
522+
<phase>generate-sources</phase>
523+
<configuration>
524+
<protoSourceRoot>${basedir}/../format/</protoSourceRoot>
525+
</configuration>
526+
<goals>
527+
<goal>compile</goal>
528+
<goal>compile-custom</goal>
529+
</goals>
530+
</execution>
531+
<execution>
532+
<id>proto-test-compile</id>
533+
<phase>generate-test-sources</phase>
534+
<goals>
535+
<goal>test-compile</goal>
536+
<goal>test-compile-custom</goal>
537+
</goals>
538+
</execution>
539+
</executions>
540+
</plugin>
509541
</plugins>
510542
</pluginManagement>
511543
</build>

0 commit comments

Comments
 (0)