You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2020/05/15 11:04:03 UTC
[GitHub] [pulsar] Mefl opened a new issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Mefl opened a new issue #6967:
URL: https://github.com/apache/pulsar/issues/6967
**Describe the bug**
The pulsar-client consume command has an option `--subscription-position` documented since 2.5.1, the class in the 2.5 branche does not have the option but it is ok on master branche. I think one part of the code wasn't merged on 2.5 branche.
**To Reproduce**
The documentation on 2.5.1 : https://pulsar.apache.org/docs/en/2.5.1/reference-cli-tools/#consume
The CmdConsume class on 2.5 branche : https://github.com/apache/pulsar/blob/branch-2.5/pulsar-client-tools/src/main/java/org/apache/pulsar/client/cli/CmdConsume.java
The CmdConsume class on master branche : https://github.com/apache/pulsar/blob/45f2d80b40523dfc8ae189ec45af44d351408d87/pulsar-client-tools/src/main/java/org/apache/pulsar/client/cli/CmdConsume.java#L85
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet edited a comment on issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Posted by GitBox <gi...@apache.org>.
Anonymitaet edited a comment on issue #6967:
URL: https://github.com/apache/pulsar/issues/6967#issuecomment-630034929
Seems that [2.5.1 doc](https://pulsar.apache.org/docs/en/2.5.1/reference-cli-tools/#consume) (which includes `--subscription-position`) is not generated based on [2.5.0 doc](https://pulsar.apache.org/docs/en/2.5.0/reference-cli-tools/) (which does not include `--subscription-position`)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Huanli-Meng commented on issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Posted by GitBox <gi...@apache.org>.
Huanli-Meng commented on issue #6967:
URL: https://github.com/apache/pulsar/issues/6967#issuecomment-652727614
@Mefl , how about the process?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] jiazhai commented on issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Posted by GitBox <gi...@apache.org>.
jiazhai commented on issue #6967:
URL: https://github.com/apache/pulsar/issues/6967#issuecomment-629906508
also @tuteng who is the release manager of 2.5.1.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Anonymitaet commented on issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on issue #6967:
URL: https://github.com/apache/pulsar/issues/6967#issuecomment-630034929
@tuteng seems that [2.5.1 doc](https://pulsar.apache.org/docs/en/2.5.1/reference-cli-tools/#consume) (which includes `--subscription-position`) is not generated based on [2.5.0 doc](https://pulsar.apache.org/docs/en/2.5.0/reference-cli-tools/) (which does not include `--subscription-position`), could u pls take a look?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] Mefl commented on issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Posted by GitBox <gi...@apache.org>.
Mefl commented on issue #6967:
URL: https://github.com/apache/pulsar/issues/6967#issuecomment-630058682
Sure, I will correct it 👍
You can assign it to me
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] jiazhai commented on issue #6967: pulsar cli option documented but the code is not in the 2.5 branche
Posted by GitBox <gi...@apache.org>.
jiazhai commented on issue #6967:
URL: https://github.com/apache/pulsar/issues/6967#issuecomment-629906423
@Mefl Thanks for the issue reporting. Looks like this is a bug in doc release. Seems 2.5.1 doc included content that should not be included.
@Mefl Would you like to help fix this doc issue?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org