Skip to content

[branch-2.6] update to stable version of bk 4.11.1 - #8604

Merged
codelipenghui merged 2 commits into
apache:branch-2.6from
jiazhai:update_bk_4111
Nov 18, 2020
Merged

[branch-2.6] update to stable version of bk 4.11.1#8604
codelipenghui merged 2 commits into
apache:branch-2.6from
jiazhai:update_bk_4111

Conversation

@jiazhai

@jiazhai jiazhai commented Nov 18, 2020

Copy link
Copy Markdown
Member

It would be great to update branch-2.6 to a stable version of bk. so some issues could be fixed.
e.g. apache/bookkeeper#2333

@sijie sijie left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's make sure the branch-2.6 is able to build

@jiazhai

jiazhai commented Nov 18, 2020

Copy link
Copy Markdown
Member Author

doing the cherry-pick for #8270

changed the lience file for branch-2.6

Motivation
Upgrade Apache BookKeeper client to version 4.11.1 and RocksDB to 6.10.2 (needed by Bookie)

Modifications
Upgrade Apache BookKeeper client, fix a few build errors, due to the introduction of BookieServiceInfo structure (see BP-38).

Verifying this change
This change is already covered by existing tests

* Upgrade Apache BookKeeper to 4.11.1

* fix build

* fix LICENSE files

* Use BK 4.11.1

* Upgrade RocksDB to 6.10.2

* Fix license and upgrade maven assembly plugin

* update license

* restore bouncy castle license

* fix licenses

* Exclude BouncyCastle inherited from BK

Co-authored-by: Enrico Olivelli <enrico.olivelli@diennea.com>
Co-authored-by: Enrico Olivelli <eolivelli@apache.org>(cherry picked from commit 465964b)
@codelipenghui
codelipenghui merged commit ba05dc1 into apache:branch-2.6 Nov 18, 2020
@vishnuvardhanp4

Copy link
Copy Markdown

how to upgrade apache pulsar 2.6.1 version to 2.7.0 without losing any data

@codelipenghui

Copy link
Copy Markdown
Contributor

@vishnuvardhanp4 If you just upgrade the broker, I think this will not lose any data. If you want to upgrade the bookkeeper too, you can do like followings:

  1. Disable the auto-recovery to prevent the not necessary data replication during the bookkeeper upgrade.(bin/bookkeeper shell autorecovery -disable)
  2. Upgrade the bookies one by one(after one upgraded, check the bookies works as expected, can write data and read data)
  3. After all bookies upgraded, enabled the auto-recovery(bin/bookkeeper shell autorecovery -enable)

@vishnuvardhanp4

vishnuvardhanp4 commented Mar 4, 2021 via email

Copy link
Copy Markdown

@codelipenghui

Copy link
Copy Markdown
Contributor

For the standalone mode, you just need to replace the lib directory with the new lib directory of 2.7.0 and then restart the server. You can use a separate environment to verify. Before replacing, you can backup the old lib directory.

@vishnuvardhanp4

vishnuvardhanp4 commented Mar 5, 2021 via email

Copy link
Copy Markdown

@codelipenghui

Copy link
Copy Markdown
Contributor

Yes, the data will be safe. The standalone contains the broker and bookkeeper, so the bookkeeper will also upgrade when you upgrade the standalone.

@vishnuvardhanp4

vishnuvardhanp4 commented Mar 5, 2021 via email

Copy link
Copy Markdown

@vishnuvardhanp4

vishnuvardhanp4 commented Mar 6, 2021 via email

Copy link
Copy Markdown

@eolivelli

Copy link
Copy Markdown
Contributor

If you are using pulsar standalone for development and you can lose your data you can erase the data directory and start from scratch

@vishnuvardhanp4

vishnuvardhanp4 commented Mar 6, 2021 via email

Copy link
Copy Markdown

@eolivelli

Copy link
Copy Markdown
Contributor

@vishnuvardhanp4 can you please paste more logs ?
"Failed to start pulsar" is not enough.

Side note:
Usually people use "standalone" for development, because with a single node you do not have fault tolerance.

@eolivelli

Copy link
Copy Markdown
Contributor

can you please also create a dedicated ticket "issue", this way we can follow up the discussion about your case

@vishnuvardhanp4

vishnuvardhanp4 commented Mar 8, 2021 via email

Copy link
Copy Markdown

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.

5 participants