Skip to content

Commit b34da59

Browse files
committed
parquet-hadoop to use separate JVM propertied not to clash with existing CI
1 parent ce8533d commit b34da59

File tree

1 file changed

+8
-2
lines changed

1 file changed

+8
-2
lines changed

parquet-hadoop/src/test/java/org/apache/parquet/hadoop/InterOpTester.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -38,8 +38,14 @@ public class InterOpTester {
3838
private static final Logger LOG = LoggerFactory.getLogger(InterOpTester.class);
3939
// since PARQUET_TESTING_REPO might be beyond a web proxy ...
4040
private static OkHttpClient createOkHttpClientOptProxy() {
41-
String proxyHost = System.getProperty("https.proxyHost");
42-
String proxyPort = System.getProperty("https.proxyPort");
41+
/* We use a different JVM property set,
42+
* because CI may define JVM properties
43+
* "https.proxyHost" and "https.proxyPort"
44+
* and that proxy won't support some compressions
45+
* (e.g. gzip/snappy on github.com CI).
46+
/
47+
String proxyHost = System.getProperty("parquet.https.proxyHost");
48+
String proxyPort = System.getProperty("parquet.https.proxyPort");
4349
OkHttpClient client = null;
4450
if (proxyHost != null || proxyPort != null) {
4551
try {

0 commit comments

Comments
 (0)