Github user interma commented on the issue:

    https://github.com/apache/incubator-hawq/pull/1376
  
    @superbobry Thanks for your PR and sorry for response late.
    
    * hdfsPread, LGTM, thanks for your tests.
    * hdfsSync, LGTM
    Looks you just added a new `hdfsHSync()` function and didn't change 
original `hdfsSync()` behavior, so it works with HAWQ.
    * hdfsFlush
    Do you mean you want to modify the `hdfsFlush()` behavior? I greped code, 
HAWQ just uses `hdfsHFlush()` by now. So you can feel free to modify 
`hdfsFlush()`, but make sure pass all tests (run all tests is a hard thing for 
new committer, if your modification is simple, I can run this test for you).



---

Reply via email to