vbhanuchander-lang opened a new pull request, #7909:
URL: https://github.com/apache/hop/pull/7909

   Closes #7250
   
   @hansva you asked on that issue for this to be documented ("I don't think it 
is properly documented, let's keep this ticket to do that"). While writing it I 
found the docs were not silent on UNC — they claimed the opposite:
   
   ```asciidoc
   Where `absolute-path` is a valid absolute file name for the local platform.
   UNC names are supported under Windows.
   ```
   
   That sentence is accurate about **Apache VFS**, which the surrounding table 
is taken from ("The table below lists the file system types provided by the 
default Apache VFS implementation"), but not about Hop. So a user following the 
docs had every reason to expect `\\some.network.folder\my_excel.xlsx` to work, 
which is how #7250 happened.
   
   Two changes:
   
   **1. Correct the claim** rather than leave a true-of-upstream statement that 
misleads in context, pointing at the new section instead.
   
   **2. Add a `Windows UNC paths` section** covering what the issue actually 
revealed:
   
   - UNC paths are accepted by Apache VFS but are not reliably supported in Hop
   - **why it looks inconsistent** — the reporter's confusion was that Excel 
Input failed while another transform read the same file. Different transforms 
reach the file system by different routes, so "it works over there" is not 
evidence it will work here. That seemed worth stating, since it is the part 
that costs people time.
   - a warning against relying on it in unattended runs
   - the mounted-drive workaround you recommended, for Windows and for 
Linux/macOS, with a variable example so the same pipeline is portable
   
   Docs only, no code. Cross-reference uses the `<<id, Text>>` form to match 
the rest of the manual, and `**bold**` as used elsewhere in these pages.
   
   If you would rather this were phrased as a flat "not supported" with no 
explanation of the inconsistency, say so and I will trim it — I erred toward 
explaining the failure mode because that is what the reporter was missing.


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

Reply via email to