tballison commented on code in PR #4853:
URL: https://github.com/apache/solr/pull/4853#discussion_r3937581684


##########
solr/modules/extraction/src/test/org/apache/solr/handler/extraction/TikaServerExtractionBackendTest.java:
##########
@@ -124,24 +125,24 @@ public void testPdfWithImageRecursive() throws Exception {
     try (TikaServerExtractionBackend backend =
         new TikaServerExtractionBackend(tikaContainer.getBaseUrl())) {
       byte[] data = 
Files.readAllBytes(getFile("extraction/pdf-with-image.pdf"));
-      // Enable recursive extraction and set header to extract images from PDF
+      // Tika 4.x removed the X-Tika-* header family entirely (see 
resolveConfigJson's javadoc);
+      // there is no replacement for this combination. Per-request config now 
requires the
+      // multipart /config endpoints, but Tika 4.x has no XML-output variant 
of /rmeta/config, so

Review Comment:
   out of date comment



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to