Skip to content

[Doc] Add descriptions for C++ Client API doc (ClientConfiguration.h + ConsumerConfiguration.h + ReaderConfiguration.h)#10103

Merged
Anonymitaet merged 4 commits intoapache:masterfrom
Anonymitaet:0331
Apr 2, 2021
Merged

[Doc] Add descriptions for C++ Client API doc (ClientConfiguration.h + ConsumerConfiguration.h + ReaderConfiguration.h)#10103
Anonymitaet merged 4 commits intoapache:masterfrom
Anonymitaet:0331

Conversation

@Anonymitaet
Copy link
Member

Fix a part of #9282

Below are previews on my local environment:

  • ClientConfiguration.h
    image

  • ConsumerConfiguration.h
    image

  • ReaderConfiguration.h
    image

@Anonymitaet Anonymitaet added the doc Your PR contains doc changes, no matter whether the changes are in markdown or code files. label Mar 31, 2021
@Anonymitaet Anonymitaet self-assigned this Mar 31, 2021
@Anonymitaet
Copy link
Member Author

@BewareMyPower @Huanli-Meng @jennifer88huang could you please help review? Thanks

@Anonymitaet Anonymitaet added this to the 2.8.0 milestone Mar 31, 2021
@BewareMyPower
Copy link
Contributor

I've left a few comments, PTAL.

@Anonymitaet
Copy link
Member Author

@BewareMyPower many thanks for your guidance! I've incorporated ur comments, PTAL.

@BewareMyPower
Copy link
Contributor

Please format your file before pushing your commits.

@Anonymitaet Anonymitaet merged commit 21adedc into apache:master Apr 2, 2021
@Anonymitaet Anonymitaet deleted the 0331 branch April 2, 2021 09:42
wangjialing218 pushed a commit to wangjialing218/pulsar that referenced this pull request Apr 9, 2021
…+ ConsumerConfiguration.h + ReaderConfiguration.h) (apache#10103)

* [Doc] Add descriptions for C++ Client API doc

* update

* update

* update

Co-authored-by: Anonymitaet <anonymitaet_hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc Your PR contains doc changes, no matter whether the changes are in markdown or code files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments