cj-zhukov commented on code in PR #19319:
URL: https://github.com/apache/datafusion/pull/19319#discussion_r2636881648


##########
datafusion-examples/examples/data_io/parquet_encrypted.rs:
##########
@@ -32,13 +33,14 @@ pub async fn parquet_encrypted() -> 
datafusion::common::Result<()> {
     // The SessionContext is the main high level API for interacting with 
DataFusion
     let ctx = SessionContext::new();
 
-    // Find the local path of "alltypes_plain.parquet"
-    let testdata = datafusion::test_util::parquet_test_data();
-    let filename = &format!("{testdata}/alltypes_plain.parquet");
+    let path = PathBuf::from(env!("CARGO_MANIFEST_DIR"))

Review Comment:
   Thanks, that makes sense.
   
   I agree that using fewer, clearer datasets in the examples is a good 
direction. I can look into rewriting this `parquet_encrypted` example to read 
from `cars.csv` and then write the encrypted parquet output, instead of relying 
on `alltypes_plain.parquet`.
   
   I’ll prototype that approach and report back once I confirm everything works 
cleanly with the encryption workflow. 



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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to