mattyb149 commented on a change in pull request #4755:
URL: https://github.com/apache/nifi/pull/4755#discussion_r560426686
##########
File path:
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-client-service/src/test/groovy/org/apache/nifi/elasticsearch/integration/ElasticSearch5ClientService_IT.groovy
##########
@@ -175,6 +177,41 @@ class ElasticSearch5ClientService_IT {
runner.assertValid()
}
+ @Test
+ void testNullSuppression() {
Review comment:
I couldn't get any of these tests to pass, looks like it expects the
`messages` index to already exist, and for things like testBasicSearch it
appears that it was supposed to be primed with 15 messages. I created an empty
`messages` index and ran just this testNullSuppression, and got the following
error (using ES 5.6.16):
```
org.apache.nifi.elasticsearch.ElasticsearchError:
org.elasticsearch.client.ResponseException: POST http://localhost:9200/_bulk:
HTTP/1.1 400 Bad Request
{"error":{"root_cause":[{"type":"action_request_validation_exception","reason":"Validation
Failed: 1: type is
missing;"}],"type":"action_request_validation_exception","reason":"Validation
Failed: 1: type is missing;"},"status":400}
```
Am I doing something wrong? Couldn't find anything in the ES docs about
ignoring the document type...
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
