Github user joewitt commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/556#discussion_r68044679
  
    --- Diff: 
nifi-nar-bundles/nifi-media-bundle/nifi-media-processors/src/test/java/org/apache/nifi/processors/image/ExtractImageMetadataTest.java
 ---
    @@ -37,7 +37,7 @@
         private static String BMP_HEADER = "BMP Header.";
         private static String JPEG_HEADER = "JPEG.";
         private static String GIF_HEADER = "GIF Header.";
    -    private static String PNG_HEADER = "PNG.";
    +    private static String PNG_HEADER = "PNG-";
    --- End diff --
    
    Ok.  Even that punts the problem because it just means we'll stick with an 
older dependency.  My vote here would be we keep it as-is.  Update the 
migration guide to call this issue out when going to 0.7 (and 1.0).  And we 
update the processor documentation to highlight that the attributes extracted 
by this processor are not stable and subject to change across upgrades.  It 
isn't great but it is honest.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to