pig-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jarek Cecho" <jar...@apache.org>
Subject Review Request 17240: PIG-3717 AvroStorage is not filtering hidden files on directory recursion
Date Thu, 23 Jan 2014 15:51:04 GMT

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/17240/
-----------------------------------------------------------

Review request for pig.


Bugs: PIG-3717
    https://issues.apache.org/jira/browse/PIG-3717


Repository: pig-git


Description
-------

We do have PATH_FILTER correctly defined to filter out hidden files, but it's used only during
recursion entry point. When doing directory listing inside the recursion, we did not used
the filter. I've added the filter there.


Diffs
-----

  contrib/piggybank/java/src/main/java/org/apache/pig/piggybank/storage/avro/AvroStorageUtils.java
7235ab8df32a902dfb9b90622e02f16c10f6a6e4 
  contrib/piggybank/java/src/test/java/org/apache/pig/piggybank/test/storage/avro/TestAvroStorageUtils.java
99bc43eb83fbf272806dc7cca66c904bcbf71577 

Diff: https://reviews.apache.org/r/17240/diff/


Testing
-------

Added new test case covering this issue.


Thanks,

Jarek Cecho


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message