Github user jackylk commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2342#discussion_r191153672
  
    --- Diff: store/search/src/main/scala/org/apache/spark/rpc/Master.scala ---
    @@ -82,7 +80,7 @@ class Master(sparkConf: SparkConf) {
               do {
                 try {
                   LOG.info(s"starting registry-service on $hostAddress:$port")
    -              val config = RpcEnvConfig(
    +              val config = CarbonReflectionUtils.getRpcEnvConfig(
    --- End diff --
    
    Is it possible to move `CarbonReflectionUtils.getRpcEnvConfig` to this 
module so that this module doesn't need to depend on spark-common module?


---

Reply via email to