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 2021/10/21 05:15:20 UTC
[GitHub] [pulsar] gaoran10 opened a new pull request #12448: [Pulsar SQL] Pulsar SQL support query big entry data
gaoran10 opened a new pull request #12448:
URL: https://github.com/apache/pulsar/pull/12448
### Motivation
Currently, the Pulsar SQL didn't set `NettyMaxFrameSizeBytes` for the BookKeeper client, so users couldn't use Pulsar SQL query big entry data.
### Modifications
Set configuration `NettyMaxFrameSizeBytes` when generating the BookKeeper client in Pulsar SQL.
### Verifying this change
Add a new test to verify using Pulsar SQL to query big entry data.
### Does this pull request potentially affect one of the following parts:
*If `yes` was chosen, please highlight the changes*
- Dependencies (does it add or upgrade a dependency): (no)
- The public API: (no)
- The schema: (no)
- The default values of configurations: (no)
- The wire protocol: (no)
- The rest endpoints: (no)
- The admin cli options: (no)
- Anything that affects deployment: (no)
### Documentation
Check the box below and label this PR (if you have committer privilege).
Need to update docs?
- [ ] `doc-required`
(If you need help on updating docs, create a doc issue)
- [x] `no-need-doc`
(Please explain why)
- [ ] `doc`
(If this PR contains doc changes)
--
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.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on pull request #12448: [Pulsar SQL] Pulsar SQL support query big entry data
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #12448:
URL: https://github.com/apache/pulsar/pull/12448#issuecomment-948266025
@gaoran10:Thanks for providing doc info!
--
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.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] eolivelli commented on pull request #12448: [Pulsar SQL] Pulsar SQL support query big entry data
Posted by GitBox <gi...@apache.org>.
eolivelli commented on pull request #12448:
URL: https://github.com/apache/pulsar/pull/12448#issuecomment-948265878
@gaoran10:Thanks for your contribution. For this PR, do we need to update docs?
(The [PR template contains info about doc](https://github.com/apache/pulsar/blob/master/.github/PULL_REQUEST_TEMPLATE.md#documentation), which helps others know more about the changes. Can you provide doc-related info in this and future PR descriptions? Thanks)
--
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.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [pulsar] codelipenghui merged pull request #12448: [Pulsar SQL] Pulsar SQL support query big entry data
Posted by GitBox <gi...@apache.org>.
codelipenghui merged pull request #12448:
URL: https://github.com/apache/pulsar/pull/12448
--
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.
To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org