Skip to content

Migrate command regenerate-interleaved-storage-index-file#2035

Merged
sijie merged 6 commits intoapache:masterfrom
zymap:command-RISIF
Apr 9, 2019
Merged

Migrate command regenerate-interleaved-storage-index-file#2035
sijie merged 6 commits intoapache:masterfrom
zymap:command-RISIF

Conversation

@zymap
Copy link
Copy Markdown
Member

@zymap zymap commented Apr 1, 2019

Descriptions of the changes in this PR:

#2033

Regenerate an interleaved storage index file, from available entrylogger files.

Usage:  bkctl bookie regenerate-interleaved-storage-index-file [flags]

Flags:

    -b, --b64password
        The password in base64 encoding, for cases where the password is not
        UTF-8.

    -d, --dryrun
        Process the entryLogger, but don't write anthing.

    -l, --ledgerids
        Ledger(s) whose index needs to be regenerated. Multiple can be
        specified, comma separated.

    -p, --password
        The bookie stores the password in the index file, so we need it to
        regenerate.This must match the value in the ledger metadata.


    -h, --help
        Display help information

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 4, 2019

run integration tests

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 8, 2019

@sijie I resolve it by pre check. PTAL

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 8, 2019

run pr validation

@sijie
Copy link
Copy Markdown
Member

sijie commented Apr 9, 2019

@zymap there is one checkstyle violation. can you check?

[ERROR] /home/jenkins/jenkins-slave/workspace/bookkeeper_precommit_pullrequest_validation/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/BookieShell.java:49:8: Unused import: java.util.Set. [UnusedImports]

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 9, 2019

run bookkeeper-server replication tests

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 9, 2019

run pr validation

@zymap
Copy link
Copy Markdown
Member Author

zymap commented Apr 9, 2019

run integration tests
run bookkeeper-server replication tests

1 similar comment
@sijie
Copy link
Copy Markdown
Member

sijie commented Apr 9, 2019

run integration tests
run bookkeeper-server replication tests

@sijie sijie added this to the 4.10.0 milestone Apr 9, 2019
@sijie sijie merged commit 7ad5849 into apache:master Apr 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants