[ 
https://issues.apache.org/jira/browse/DERBY-6845?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15039555#comment-15039555
 ] 

Bryan Pendleton commented on DERBY-6845:
----------------------------------------


Hi Rick,

Thanks for putting together the draft documentation.

Here's some feedback.

This is an interesting tool to document, because
the premise of the tool is that a Very Bad Thing
has happened, and the user is in a Serious Panic.

So we want to say some very calming words, and set
them up for success, because their emotional state
is likely to be poor when they are reading this page.

I think you have done a very good job at this,
but I think we can make the documentation still
further calming and step-by-step.

I expect that after you read my feedback you'll
want to take another swing at this, and maybe we
can toss about some ideas.

Wherever we end up, thanks again for putting the
tool together, it is VERY nice to have it on hand
for when lightning strikes.

thanks,

bryan

1) I think the word "siphon" might be hard for many
   readers, particularly non-English-speakers. Perhaps
   something like "extract" or "unload" or "recover"
   or "copy" or "retrieve"?

2) I read 'corrupt, unbootable' as "corrupt AND unbootable";
   I assume that rawDBReader can be used on "corrupt yet
   bootable" databases as well, so perhaps we should be
   explicit and say "corrupt or unbootable", or 
   "corrupt databases, even if they are unbootable"?

   Or, if the tool is really designed only for unbootable
   databases, then maybe just remove the "corrupt,"
   since an unbootable database is clearly corrupt.

3) I'd remove ", as with any database engine,", because
   who cares about anybody else :)

4) Perhaps change "This technique is not guaranteed" to
   "The rawDBReader tool is not guaranteed".

5) I'd remove "Strictly speaking,"

6) Perhaps change

        For the remainder of this section, we will
        talk about two databases:

   to

        When running the rawDBReader tool, you will 
        be working with two databases:

        corruptDB - This is the unbootable database whose
                    data you are trying to salvage.
        healthyDB - This is a new database which you
                    create in order to hold the retrieved data.

        The rawDBReader tool works by copying all
        your data from the unbootable database into
        a totally separate database.

        Each table in the corruptDB will be copied to
        a table with the same table name and column
        names in healthyDB, but the table will be
        in a separate schema so you can tell which
        is which.

7) Perhaps change the part where you say:

        To load the rawDBReader tool

   to something like:

        Using the rawDBReader tool is a three-step process:

        1) You call SYSCS_REGISTER_TOOL with the
           correct arguments, which will prepare
           the healthyDB to accept any data from the
           corruptDB, and will also generate a text
           file containing a series of commands. This
           text file is called the 'recovery script'.

        2) You then run the recovery script to retrieve
           all your data from the corruptDB into the
           healthyDB

        3) After this is done, you unload the tool by
           making another SYSCS_REGISTER_TOOL call.

        These three steps are described in more detail below.

8) In both your examples you use "CONTROL_" as the name
   of your control schema, but then your example recovery
   script uses 'S1'. That was confusing. Can that be
   made more clear?

9) All your examples are with embedded syntax. Does
   the tool work with a client-server configuration?
   If so, we should probably document what adjustments
   are necessary; if not, we should simply say something
   like:

        To run the rawDBReader tool, you must be
        signed on to the machine where the corruptDB
        resides and you must have the derby.jar and
        derbytools.jar files in your CLASSPATH.




> Document the rawDBReader optional tool
> --------------------------------------
>
>                 Key: DERBY-6845
>                 URL: https://issues.apache.org/jira/browse/DERBY-6845
>             Project: Derby
>          Issue Type: Improvement
>          Components: Documentation
>    Affects Versions: 10.13.0.0
>            Reporter: Rick Hillegas
>            Assignee: Rick Hillegas
>         Attachments: derby-6845-01-aa-firstCut.diff, 
> derby-6845-01-aa-firstCut.tar
>
>
> Document the optional tool added by DERBY-6136.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to