Skip to content

Commit 6a353f4

Browse files
committed
Update per review comments, including recommendations for FAQ and troubleshooting.
Signed-off-by: Dave Welsch <[email protected]>
1 parent 466c331 commit 6a353f4

File tree

1 file changed

+52
-27
lines changed

1 file changed

+52
-27
lines changed

analyses/0014-vitess/analysis.md

+52-27
Original file line numberDiff line numberDiff line change
@@ -145,6 +145,12 @@ works, in both the [Overview](https://vitess.io/docs/21.0/overview/) and
145145

146146
The introduction to the Overview could be more helpful to new users.
147147

148+
There is an extensive FAQ that contains much information of all types, but especially conceptual information. The overview introduction in the FAQ is actually more descriptive and succinct than the one on the main web page:
149+
150+
> Vitess is a database solution for deploying, scaling and managing large clusters of database instances.
151+
152+
The FAQ is outside the versioned documentation.
153+
148154
**Feature complete**: Is the documentation feature complete? (i.e., each product
149155
feature is documented)?
150156

@@ -161,8 +167,9 @@ use Vitess:
161167
- Setup and use
162168
- Administration
163169

164-
Except for Installation, all task documentation is in the
165-
[User Guides](https://vitess.io/docs/21.0/user-guides/).
170+
Except for Installation, all task documentation is in one of two places:
171+
- The [User Guides](https://vitess.io/docs/21.0/user-guides/).
172+
- An extensive FAQ that contains task instructions.
166173

167174
Despite being labeled "Task-based guides," the task instructions in the User
168175
Guide are written from a feature-oriented perspective, which lessens the
@@ -206,7 +213,11 @@ following sections:
206213
- Advanced > Distributed Atomic Transactions
207214
- Migration
208215

209-
There is one special-purpose FAQ, for VReplication in the Reference.
216+
There is a troubleshooting section outside the versioned documentation.
217+
218+
There is a lengthy troubleshooting section in the FAQ.
219+
220+
There is a separate special-purpose FAQ, for VReplication, in the Reference.
210221

211222
There is no glossary.
212223

@@ -229,6 +240,11 @@ utilities, and these are documented in the Reference.
229240
Yes, documentation is versioned and prominently includes the latest Stable and
230241
Development versions.
231242

243+
However, there are several important sections that are maintained outside the versioned documentation:
244+
- Troubleshooting
245+
- FAQ
246+
- Information about the release procedure
247+
232248
#### New user content
233249

234250
**Getting started entry**: Is “getting started” clearly labeled? (“Getting
@@ -294,6 +310,8 @@ localization/internationalization with regard to site directory structure?
294310

295311
Yes, there are full versions of the documentation in both English and Chinese.
296312

313+
The Chinese versions are reportedly not up to date with the latest software version.
314+
297315
**Localization framework**: Is a localization framework present?
298316

299317
Yes, there seems to be some infrastructure for multiple languages. I'm not sure
@@ -433,9 +451,9 @@ content in at least two ways:
433451
- Makes the TOC more coherent
434452
- Gives meaningful results in the site-wide Search
435453

436-
Here are three examples from the [same page]
437-
(https://vitess.io/docs/21.0/user-guides/configuration-basic/global-topo/) (the
438-
content of these sections is not considered here):
454+
Here are three examples from the
455+
[same page](https://vitess.io/docs/21.0/user-guides/configuration-basic/global-topo/)
456+
(the content of these sections is not considered here):
439457

440458
- _Choosing a TopoRoot_: A good title. Describes the task ("choosing") and the
441459
object of the task ("TopoRoot").
@@ -452,15 +470,17 @@ Also on the Global TopoServer page, by the way:
452470
> The following command line options are required for every Vitess component:
453471
>
454472
> ````--topo_implementation=etcd2 --topo_global_server_address=<comma_separated_addresses>
455-
> --topo_global_root=/vitess/global```
473+
> --topo_global_root=/vitess/global
474+
> ````
456475
>
457476
> To avoid repetition we will use <topo_flags> in our examples to signify the above
458477
> flags.
459-
> ````
460478
461-
Remove this. The "<topo_flags>" placeholder does not seem to have been
462-
implemented. There are no mentions of "<topo_flags"> elsewhere in the
463-
documentation, and in any case each would have to refer back to this page.
479+
Remove this. The "<topo_flags>" placeholder does not seem to have been implemented.
480+
There are no mentions of "<topo_flags"> elsewhere in the documentation, and in any
481+
case each would have to refer back to this page.
482+
483+
In the FAQ: Rename the questions into concise headings when you consolidate the FAQ into the versioned documentation (see *Reorganize* below).
464484
465485
###### Rewrite
466486
@@ -475,14 +495,13 @@ For example, look at
475495
476496
This page appears to be well named ("Creating a Backup" is a descriptive task
477497
title) and properly placed (it resides in a logical location in the User Guide).
478-
However, the page itself doesn't contain an actual command or set of steps to
479-
back up a database: There is no actual command line given between
480-
_Configuration_ and _Common Errors and Resolutions_ The page's author might
481-
assume that it's implied, but it should be presented explicitly as a step in the
482-
procedure.
498+
However, the page itself buries the lede (it doesn't provide the procedure until
499+
far down the page): There is no actual command line given between
500+
_Configuration_ and _Common Errors and Resolutions_ The "how to" procedure should
501+
be prominently featured.
483502
484503
Further down the page, another backup option, _Using mysqlshell_, has the same
485-
shortcomings: No actual command is presented.
504+
shortcomings: No actual command is apparent.
486505
487506
Instead, write these pages as step-by-step descriptions of how to perform a
488507
task. Each step should be a concise instruction, with any required instruction
@@ -584,18 +603,24 @@ _Application programmer_:
584603
to any tasks that are identical)
585604
- Query optimization
586605
587-
(Again, these are my understanding of the Vitess user roles. Adjust the details
588-
if they're different.
606+
(Again, these are my understanding of the Vitess user roles. Adjust the details if
607+
they're different.)
608+
609+
Get rid of the FAQ. This FAQ has apparently grown from a repository for common questions into a separate de facto technical documentation set. Move all of the FAQ topics and information into the versioned documentation.
610+
611+
Merge information with existing sections and/or move FAQ topics to the documentation intact, as appropriate. For example, much of the FAQ amounts to a good conceptual overview of the product; it should be part of the conceptual overview. In its current form it is unversioned and hard to find.
612+
589613
590614
###### Troubleshooting
591615
592-
Consolidate all troubleshooting information into a Troubleshooting Guide. Or,
593-
create one troubleshooting guide per user guide/user role. In either case,
594-
create an escalation path for any problems with a task (the escalation path
595-
might be: _troubleshooting procedure > Slack Channel > project Issue_). Get rid
596-
of the
597-
[VReplication FAQ](https://vitess.io/docs/21.0/reference/vreplication/faq/) in
598-
the reference section and put the information in a troubleshooting section.
616+
Consolidate all troubleshooting information into a Troubleshooting Guide. Or, create
617+
one troubleshooting guide per user guide/user role. In either case, create an escalation
618+
path for any problems with a task (the escalation path might be:
619+
*troubleshooting procedure > Slack Channel > project Issue*). Get rid of the
620+
[VReplication FAQ](https://vitess.io/docs/21.0/reference/vreplication/faq/)
621+
in the reference section and put the information in a troubleshooting section.
622+
623+
Locate the troubleshooting guide or guides within the versioned technical documentation. Changes to the product affect troubleshooting procedures as well as other types of information, and the documentation structure should reflect that.
599624
600625
###### Glossary
601626
@@ -896,7 +921,7 @@ gracefully when magnified in the browser.
896921
**Case studies**: Are there case studies available for the project and are they
897922
documented on the website?
898923
899-
No, I did not see any case studies. Some of them many videos might contain case
924+
No, I did not see any case studies. Some of the many videos might contain case
900925
studies, though.
901926
902927
**Testimonials**: Are there user testimonials available?

0 commit comments

Comments
 (0)