viirya commented on code in PR #256:
URL: 
https://github.com/apache/arrow-datafusion-comet/pull/256#discussion_r1560378512


##########
README.md:
##########
@@ -127,6 +127,14 @@ Comet shuffle feature is disabled by default. To enable 
it, please add related c
 Above configs enable Comet native shuffle which only supports hash partiting 
and single partition.
 Comet native shuffle doesn't support complext types yet.
 
+Depending on your deployment mode you may also need to set the driver & 
executor class path(s) to explicitly contain Comet otherwise Spark may use a 
different class-loader for the Comet components than its internal components 
which will then fail at runtime. For example:
+
+```
+--driver-class-path spark/target/comet-spark-spark3.4_2.12-0.1.0-SNAPSHOT.jar

Review Comment:
   Maybe also add executor configuration alongside? Although it is trivial but 
as we already go to update this.



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to