Skip to content

Commit

Permalink
[ISSUE #1644]♻️Refactor PollingInfoProcessor#process_request method s…
Browse files Browse the repository at this point in the history
…ignature🔥
  • Loading branch information
mxsm committed Dec 7, 2024
1 parent c9e9c1d commit 85cdedb
Showing 1 changed file with 7 additions and 3 deletions.
10 changes: 7 additions & 3 deletions rocketmq-broker/src/processor/polling_info_processor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -14,18 +14,22 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
use rocketmq_remoting::code::request_code::RequestCode;
use rocketmq_remoting::net::channel::Channel;
use rocketmq_remoting::protocol::remoting_command::RemotingCommand;
use rocketmq_remoting::runtime::connection_handler_context::ConnectionHandlerContext;

#[derive(Default)]
pub struct PollingInfoProcessor {}

impl PollingInfoProcessor {
fn process_request(
&self,
pub async fn process_request(
&mut self,
_channel: Channel,

Check warning on line 28 in rocketmq-broker/src/processor/polling_info_processor.rs

View check run for this annotation

Codecov / codecov/patch

rocketmq-broker/src/processor/polling_info_processor.rs#L26-L28

Added lines #L26 - L28 were not covered by tests
_ctx: ConnectionHandlerContext,
_request_code: RequestCode,

Check warning on line 30 in rocketmq-broker/src/processor/polling_info_processor.rs

View check run for this annotation

Codecov / codecov/patch

rocketmq-broker/src/processor/polling_info_processor.rs#L30

Added line #L30 was not covered by tests
_request: RemotingCommand,
) -> RemotingCommand {
) -> crate::Result<Option<RemotingCommand>> {

Check warning on line 32 in rocketmq-broker/src/processor/polling_info_processor.rs

View check run for this annotation

Codecov / codecov/patch

rocketmq-broker/src/processor/polling_info_processor.rs#L32

Added line #L32 was not covered by tests
todo!()
}
}

0 comments on commit 85cdedb

Please sign in to comment.