Github user wzhfy commented on the issue:

    https://github.com/apache/spark/pull/20611
  
    @sujith71955 
    1. It's a little difficult to read as the pictures have different 
resolution. Maybe you can use ``` to include test results? I think this is more 
readable. For example:
    ```
    91-206:~ # hadoop fs -ls /wzh/t1
    Found 1 items
    -rwxr-xr-x   3 root supergroup         20 2017-01-13 09:53 
/wzh/t1/part-00000
    ```
    
    2. Can you also show the table data by select command, to make sure files 
are really loaded?
    3. Please add test cases for '?' pattern.
    4. Please move the test results for your fix to the section **How was this 
patch tested?**.
    5. Update the PR title as "Support wildcard in HDFS path for load table 
command"?



---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to