prateekm opened a new pull request, #1606: URL: https://github.com/apache/samza/pull/1606
Issue: Current validation for DirIndex checks if the same file _name_ is present in both filesAdded and filesRemoved. For mutable files like CURRENT this is valid since the metadata and blob ids are different. Changes: Fixed the check for files to compare FileIndex instead of file name, which accounts for the blobs and metadata associated with the file too. Also added a similar check for all blobs added and removed in the DirIndex too. Made both checks recursive to account for subdirs. API/usage changes: None Tests: Added a couple of unit tests. -- 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]
