bvaradar commented on a change in pull request #790:  Update docker_demo based 
on new bundling
URL: https://github.com/apache/incubator-hudi/pull/790#discussion_r303634878
 
 

 ##########
 File path: docs/docker_demo.md
 ##########
 @@ -335,7 +335,7 @@ running in spark-sql
 
 ```
 docker exec -it adhoc-1 /bin/bash
-$SPARK_INSTALL/bin/spark-shell --jars $HUDI_SPARK_BUNDLE --master local[2] 
--driver-class-path $HADOOP_CONF_DIR --conf 
spark.sql.hive.convertMetastoreParquet=false --deploy-mode client  
--driver-memory 1G --executor-memory 3G --num-executors 1  --packages 
com.databricks:spark-avro_2.11:4.0.0
+$SPARK_INSTALL/bin/spark-shell --jars $HUDI_SPARK_BUNDLE,$HUDI_HADOOP_BUNDLE 
--master local[2] --driver-class-path $HADOOP_CONF_DIR --conf 
spark.sql.hive.convertMetastoreParquet=false --deploy-mode client  
--driver-memory 1G --executor-memory 3G --num-executors 1 --packages 
com.databricks:spark-avro_2.11:4.0.0  
 
 Review comment:
   What classes are missing in Spark_bundle ? Wanted to see if we can avoid 
hadoop_bundle ?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to