Github user cestella commented on a diff in the pull request:
https://github.com/apache/metron/pull/1011#discussion_r187757615
--- Diff:
metron-platform/metron-data-management/src/main/java/org/apache/metron/dataloads/nonbulk/geo/GeoEnrichmentLoader.java
---
@@ -146,7 +156,14 @@ protected void loadGeoIpDatabase(CommandLine cli)
throws IOException {
System.out.println("Retrieving GeoLite2 archive");
String url = GeoEnrichmentOptions.GEO_URL.get(cli,
"http://geolite.maxmind.com/download/geoip/database/GeoLite2-City.mmdb.gz";);
String tmpDir = GeoEnrichmentOptions.TMP_DIR.get(cli, "/tmp") + "/";
// Make sure there's a file separator at the end
- File localGeoFile = downloadGeoFile(url, tmpDir);
+ int numRetries =
Integer.parseInt(GeoEnrichmentOptions.RETRIES.get(cli, "2"));
--- End diff --
can we make "2" a constant?
---
