http187 fixup

Signed-off-by: davidradl <[email protected]>

Author: davidradl

README.md

@@ -581,6 +581,7 @@ Notes:
 | connector                                                                            | required | The Value should be set to _rest-lookup_                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                                         |
 | format                                                                               | required | Flink's format name that should be used to decode REST response, Use `json` for a typical REST endpoint.                                                                                                                                                                                                                                                                                                                                                                                                                                                         |
 | url                                                                                  | required | The base URL that should be use for GET requests. For example _http://localhost:8080/client_                                                                                                                                                                                                                                                                                                                                                                                                                                                                     |
+| asyncPolling                                                                         | optional | true/false - determines whether Async Polling should be used. Mechanism is based on Flink's Async I/O.                                                                                                                                                                                                                                                                                                                                                                                                                                                           |
 | gid.connector.http.logging.level                                                     | optional | Logging levels for HTTP content. Valid values are `MIN` (the default), `REQRESPONSE` and `MAX`.                                                                                                                                                                                                                                                                                                                                                                                                                                                                  |
 | lookup-method                                                                        | optional | GET/POST/PUT (and any other) - determines what REST method should be used for lookup REST query. If not specified, `GET` method will be used.                                                                                                                                                                                                                                                                                                                                                                                                                    |
 | lookup.cache                                                                         | optional | Enum possible values: `NONE`, `PARTIAL`. The cache strategy for the lookup table. Currently supports `NONE` (no caching) and `PARTIAL` (caching entries on lookup operation in external API).                                                                                                                                                                                                                                                                                                                                                                    |

src/main/java/com/getindata/connectors/http/internal/sink/httpclient/PerRequestSubmitter.java

@@ -28,6 +28,7 @@ public PerRequestSubmitter(
             HttpClient httpClient) {
 
         super(properties, headersAndValues, httpClient);
+
     }
 
     @Override

src/main/java/com/getindata/connectors/http/internal/table/lookup/BodyBasedRequestFactory.java

@@ -49,8 +49,6 @@ protected Builder setUpRequestMethod(LookupQueryInfo lookupQueryInfo) {
         builder
             .uri(constructUri(lookupQueryInfo))
             .method(methodName, BodyPublishers.ofString(body));
-        // we do not view the lookup keys as sensitive; therefore the request body is
-        // not obfuscated.
         HttpLogger.getHttpLogger(options.getProperties()).logRequestBody(body);
         return builder;
     }