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

    https://github.com/apache/incubator-flink/pull/65#discussion_r16260290
  
    --- Diff: 
stratosphere-java/src/test/java/eu/stratosphere/api/java/io/CsvInputFormatTest.java
 ---
    @@ -423,5 +423,42 @@ private void testRemovingTrailingCR(String 
lineBreakerInFile, String lineBreaker
                        fail("Test erroneous");
                }
        }
    +   
    +   @Test
    +   public void testSkipHeader() throws IOException {
    +           try {
    +                   final String fileContent = 
"HEAD\nHEAD\n111|222|333|444|555|\n666|777|888|999|000|\n";
    +                   final FileInputSplit split = 
createTempFile(fileContent);       
    --- End diff --
    
    If you use a test that creates 8 splits of size 6, then it will fail, 
because some header lines are in the second split.


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