fix: change polling strategy to every 5 sec for doctrans

Author: JanEbbing

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## Unreleased 
+### Fixed
+* Changed document translation to poll the server every 5 seconds. This should greatly reduce observed document translation processing time.
+
+
 ## [1.2.0] - 2023-03-22
 ### Added
 * Script to check our source code for license headers and a step for them in the CI.
@@ -78,6 +83,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 Initial version.
 
 
+[Unreleased]: https://github.com/DeepLcom/deepl-java/compare/v1.2.0...HEAD
 [1.2.0]: https://github.com/DeepLcom/deepl-java/compare/v1.1.0...v1.2.0
 [1.1.0]: https://github.com/DeepLcom/deepl-java/compare/v1.0.1...v1.1.0
 [1.0.1]: https://github.com/DeepLcom/deepl-java/compare/v1.0.0...v1.0.1

deepl-java/src/main/java/com/deepl/api/Translator.java

@@ -3,9 +3,6 @@
 // license that can be found in the LICENSE file.
 package com.deepl.api;
 
-import static java.lang.Math.max;
-import static java.lang.Math.min;
-
 import com.deepl.api.http.HttpResponse;
 import com.deepl.api.http.HttpResponseStream;
 import com.deepl.api.parsing.Parser;
@@ -1026,11 +1023,7 @@ private void checkResponse(
   }
 
   private int calculateDocumentWaitTimeMillis(Long secondsRemaining) {
-    if (secondsRemaining != null) {
-      double secs = ((double) secondsRemaining) / 2.0 + 1.0;
-      secs = max(1.0, min(secs, 60.0));
-      return (int) (secs * 1000);
-    }
-    return 1000;
+    // secondsRemaining is currently unreliable, so just poll equidistantly
+    return 5000;
   }
 }