[Doc] Add descriptions for bookieID#9779
Conversation
Anonymitaet
added
the
doc
|
@eolivelli could you please help review? Thanks Is the option |
|
@eolivelli could you please help review? Thanks |
eolivelli
left a comment
eolivelli
left a comment
There was a problem hiding this comment.
Thank you very much.
Overall it looks great.
I left a little suggestion
Huanli-Meng
left a comment
Huanli-Meng
left a comment
There was a problem hiding this comment.
Leave my comments. PTAL
|
|
||
| # If you want to custom bookie ID or use a dynamic network address for the bookie, | ||
| # you can set this option. | ||
| # Bookie advertises itself using bookieId rather than |
There was a problem hiding this comment.
| # Bookie advertises itself using bookieId rather than | |
| # Bookie advertises itself using the `bookieId` rather than |
|
/pulsarbot run-failure-checks |
|
@eolivelli I've merged master and re-run the test many times but it still fails, I do not know how to deal with the test errors, could you please take a look? Many thanks! |
|
/pulsarbot run-failure-checks |
|
This change is only about docs, tests shouldn't even run |
|
@eolivelli thanks. I'm wondering that since in this PR, I also modify the |
|
good catch ! |
6 participants
Add doc for #9174 and #9019
Update master doc firstly and then update 2.7.x.