Attila Bukor has posted comments on this change.

Change subject: kudu-spark: change default master address to local fqdn
......................................................................


Patch Set 2: -Code-Review

(2 comments)

http://gerrit.cloudera.org:8080/#/c/8072/2/java/kudu-spark-tools/src/main/scala/org/apache/kudu/spark/tools/ImportExportFiles.scala
File 
java/kudu-spark-tools/src/main/scala/org/apache/kudu/spark/tools/ImportExportFiles.scala:

Line 24: import org.apache.spark.sql.SparkSession
> never mind, checked it in the meantime, it was unused
Done


PS2, Line 38: InetAddress.getLocalHost.getCanonicalHostName
> We could change the method to be 
I believe that would happen if the local hostname is not added to the 
/etc/hosts file, but let me try to reproduce it. I don't think that can happen 
otherwise.


-- 
To view, visit http://gerrit.cloudera.org:8080/8072
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I10ec7414c451f54b95d86663d743162688e304ba
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to