kou commented on PR #43906: URL: https://github.com/apache/arrow/pull/43906#issuecomment-2325429731
I think that we can't inject Parquet test data directory path by environment variable. Because environment variable doesn't exist on Web browser. But... we may be able to use https://github.com/pyodide/pyodide/blob/a448c045cf5640c5050e0c3722bafc0d58659fcc/src/js/pyodide.ts#L151-L160 : ```diff diff --git a/python/scripts/run_emscripten_tests.py b/python/scripts/run_emscripten_tests.py index 1a4b4a4e05..79996b1f92 100644 --- a/python/scripts/run_emscripten_tests.py +++ b/python/scripts/run_emscripten_tests.py @@ -19,6 +19,7 @@ import argparse import contextlib import http.server +import json import os import queue import shutil @@ -153,7 +154,7 @@ class NodeDriver: self.execute_js( f""" const {{ loadPyodide }} = require('{dist_dir}/pyodide.js'); - let pyodide = await loadPyodide(); + let pyodide = await loadPyodide({json.dumps({"PARQUET_TEST_DATA": os.environ.get("PARQUET_TEST_DATA")})}); """ ) ``` -- 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]
