Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate the loose application from WebSphere Liberty doc to OL.io doc #6852

Open
gkwan-ibm opened this issue Sep 6, 2023 · 7 comments
Open

Comments

@gkwan-ibm
Copy link
Member

Migrate the loose application from WebSphere Liberty doc to OL.io doc

@ramkumar-k-9286
Copy link
Contributor

Hi David @dmuelle

A draft document has been added for loose applications:

Link: https://docs-draft-openlibertyio.mqj6zf7jocq.us-south.codeengine.appdomain.cloud/docs/latest/loose-applications.html

Please review the same.

Regards,
Ramkumar.

@dmuelle
Copy link
Member

dmuelle commented Oct 19, 2023

@ramkumar-k-9286 - some initial feedback-

  • I dont think we need Normal/Loose Application headings, or the example list in the Normal section. Just keep the first sentence in that section and combine it with the Loose applications section make an intro with no other heading.
  • run time ---> runtime
  • dont mention WebSphere Application Server Developer Tools
  • use definition list formatting for the two ways to provide xml file
  • link to the server package command doc
  • try to combine the two lists in the config file section- put the examples with the corresponding actions
  • dont enclose the elements in <brackets> in the text
  • link to the config element docs for each element on first mention
  • add noun descriptors for all code entities in the text
  • run acrolinx

I'll wait on further comments until we have dev review-

@dmuelle
Copy link
Member

dmuelle commented Oct 19, 2023

one thing that is not clear is where the config file exists and what it's relationship to the server.xml is- how does the runtime find the loose app?

@dmuelle dmuelle modified the milestones: 23.0.0.11, 23.0.0.12 Nov 8, 2023
@dmuelle dmuelle modified the milestones: 23.0.0.12, 24.0.0.1 Dec 6, 2023
@dmuelle dmuelle modified the milestones: 24.0.0.1, 24.0.0.2 Jan 25, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.2, 24.0.0.3 Feb 22, 2024
@dmuelle dmuelle removed this from the 24.0.0.3 milestone Mar 26, 2024
@dmuelle dmuelle added this to the 24.0.0.5 milestone Apr 29, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.5, 24.0.0.6 May 17, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.6, 24.0.0.7 Jun 13, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.7, 24.0.0.8 Jul 12, 2024
ramkumar-k-9286 added a commit that referenced this issue Jul 30, 2024
6852-draft doc for loose applications-comm-inc-1

#6852
@ramkumar-k-9286
Copy link
Contributor

ramkumar-k-9286 commented Aug 6, 2024

Additional comments received on Slack

Would adding a sentence after "Normally .." be helpful? That would create a parallal to "Normally", adding balance to the paragraph:
"Normally an application is contained under one directory or in one archive, with its content, modules, resources, classdata, and metadata at known locations within that directory.
(Added)
"Alternatively, using loose applications, the content of an application may be split between multiple physical locations."
(I'm not sure if "Alternatively" is necessary.)


For "When you run the server package command", does this apply to both ways to provide the XML file? Having indentation groups the statement with "Using the application dropins folder".
This switches between passive and active ("when using" vs "you can use"):
"When using a loose application, the Open Liberty server uses a loose application configuration to locate application content, rather than locating the application content from a root directory or root archive. You can use the loose application configuration to locate application content in the following ways:"
I'd change:
"Map a Java bin/output folder that is not in the usual location into the WEB-INF/classes folder. This location might be in a different folder due to your workspace preferences, corporate guidelines, source control project layout guidelines, and so on. You might have multiple Java source and output locations in the same project, and want to map them both to WEB-INF/classes."
To:
"Map one or more bin/output folders onto the WEB-INF/classes folder. A mapped folder may needed, for example, because of workspace preferences, coporate guidelines, or source control project layout guidelines. Multiple mapped folders may be needed because a project has multiple output locations, or because multiple projects are be mapped onto the same WEB-INF/classes folder."
Note the change of "into" to "onto": The contents of the bin/output folder become the content of the WEB-INF/classes folder. "output" does NOT become a child folder of WEB-INF/classes.
I'd change:
"A separate Java project that you want to treat like a JAR file in WEB-INF/lib."
To:
"The output of a Java project, packaged as a JAR file, and created at a physical location outside of the virtual application."
I'd drop the sections with "Configred Application" and "Dropins Application" (but leave the NOTE!) The NOTE seems a sufficient summary of the examples that were provided.
Maybe change "The following code is" to "The following is" or "Here is"? Calling the XML text "code" is confusing to me. (There are two occurrences.)
Under "Files", "sourceOnDisk`" has a stray apostrophe.
I'd remove the text ", which are resolved correctly".
Maybe use "the physical file name" instead of "the actual name on the disk"
Replace:
"If you have two folders with the same name, the same virtual location in the loose application configuration,"
With:
"If you have two folders mapped to the same virtual location in the loose application configuration,"
A "location" includes both the path to the location and the simple name of the location.
I'd clarify how the file is "wrong", changing:
"If the first file found is the wrong file,"
To:
"If the first file found is the wrong file to be used,"
Or maybe remove "wrong"?
Also, I'd move that sentence to the preceding paragraph, starting the second paragraph with the sentence:
"The first occurrence applies to files defined in the dir elements"

Maybe change:
"could display unexpected behavior."
To:
"could behave unexpectedly."
"display" seems to centered on visible behavior.
The text "in web applications" is unnecessary in:
"However, if you use ServletContext.getRealPath in web applications to obtain a"
Maybe, change "applications" to "web applications" in the first sentence:
"You can use ServletContext.getRealPath in your web applications"

I'd change:
"The ServletContext.getRealPath allows only a single physical path to be returned, and the loose application might have merged multiple directories to form one path visible to the application."
To:
"When using loose applications, ServletContext.getRealPath is unreliable for accessing physical files. Loose applications might merge multiple directories to provide content at one virtual path visible to the application, and ServletContext.getRealPath can provide only one of the mapped physical paths."
Maybe add a bold CAUTION here?

ramkumar-k-9286 added a commit that referenced this issue Aug 6, 2024
6852-draft doc for loose applications-comm-inc-4

#6852
@ramkumar-k-9286
Copy link
Contributor

Hi Thomas @tbitonti

The suggested comments have been incorporated.

The draft document Link: https://docs-draft-openlibertyio.mqj6zf7jocq.us-south.codeengine.appdomain.cloud/docs/latest/loose-applications.html

Please review the same and add the Developer Reviewed label if satisfied with the document.

Please feel free to add comments here if you need me to make any changes to the document.

Regards,
Ramkumar.

ramkumar-k-9286 added a commit that referenced this issue Aug 6, 2024
6852-draft doc for loose applications-comm-inc-5

#6852
ramkumar-k-9286 added a commit that referenced this issue Aug 7, 2024
6852-draft doc for loose applications-comm-inc-6

#6852
ramkumar-k-9286 added a commit that referenced this issue Aug 8, 2024
6852-draft doc for loose applications-comm-inc-7

#6852
ramkumar-k-9286 added a commit that referenced this issue Aug 8, 2024
6852-draft doc for loose applications-comm-inc-8

#6852
@dmuelle dmuelle modified the milestones: 24.0.0.8, 24.0.0.9 Aug 8, 2024
@dmuelle dmuelle modified the milestone: 24.0.0.9 Aug 29, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.9, 24.0.0.10 Sep 10, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.10, 24.0.0.11 Oct 2, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.11, 24.0.0.12 Oct 31, 2024
@dmuelle dmuelle modified the milestones: 24.0.0.12, 25.0.0.1 Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants