You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-issues@jackrabbit.apache.org by "Francesco Mari (JIRA)" <ji...@apache.org> on 2017/02/02 09:21:51 UTC

[jira] [Commented] (OAK-5275) The check command should accept the path to the store as a positional argument

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

Francesco Mari commented on OAK-5275:
-------------------------------------

The patch looks good to me, but you should also update the documentation at {{oak-doc/src/site/markdown/nodestore/segment/overview.md}} accordingly.

> The check command should accept the path to the store as a positional argument
> ------------------------------------------------------------------------------
>
>                 Key: OAK-5275
>                 URL: https://issues.apache.org/jira/browse/OAK-5275
>             Project: Jackrabbit Oak
>          Issue Type: Improvement
>          Components: run, segment-tar
>            Reporter: Francesco Mari
>            Assignee: Andrei Dulceanu
>            Priority: Minor
>              Labels: tooling
>             Fix For: 1.8, 1.6.1
>
>         Attachments: OAK-5275.patch
>
>
> The {{check}} tool requires the path to the store to be specified. The path is passed to the tool via a required option {{--path}}. This way of specifying the path to the store is verbose for no good reason. It would be nicer if the path to the Segment Store would be specified via a positional argument instead.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)