Skip to content

Commit

Permalink
[SPARK-40718][CONNECT] Replace grpc-netty-shaded withgrpc-netty
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

This patch replaces the shaded version of Netty in GRPC with the unshaded one since we're shading the result again and want to avoid double shaded package names.

### Why are the changes needed?
Unnecessary double shaded package names.

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Existing tests will check for compilation break.

Closes apache#38179 from grundprinzip/spark-40718.

Authored-by: Martin Grund <martin.grund@databricks.com>
Signed-off-by: Dongjoon Hyun <dongjoon@apache.org>
  • Loading branch information
grundprinzip authored and dongjoon-hyun committed Oct 10, 2022
1 parent 357193d commit cf05383
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion connector/connect/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@
</dependency>
<dependency>
<groupId>io.grpc</groupId>
<artifactId>grpc-netty-shaded</artifactId>
<artifactId>grpc-netty</artifactId>
<version>${io.grpc.version}</version>
</dependency>
<dependency>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import scala.collection.JavaConverters._
import com.google.common.base.Ticker
import com.google.common.cache.CacheBuilder
import io.grpc.{Server, Status}
import io.grpc.netty.shaded.io.grpc.netty.NettyServerBuilder
import io.grpc.netty.NettyServerBuilder
import io.grpc.protobuf.services.ProtoReflectionService
import io.grpc.stub.StreamObserver

Expand Down

0 comments on commit cf05383

Please sign in to comment.