[feat](protocol)extract protocol definitions (part 1) by CalvinKirs · Pull Request #60355 · apache/doris

CalvinKirs · 2026-01-29T08:45:21Z

What problem does this PR solve?

Overview

This document describes Apache Doris multi-protocol support. The architecture uses SPI
(Service Provider Interface) to decouple protocol implementations from the kernel.

Core Design

SPI Loading Flow

┌─────────────────────────────────────────────────────────────────────────┐
│ QeService (fe-core) │
│ │
│ 1. Build ProtocolConfig (from Config + FrontendOptions) │
│ 2. ProtocolLoader.loadConfiguredProtocols(config) │
│ 3. Set protocol acceptor callbacks │
│ 4. handler.start() to launch protocol servers │
└────────────────────────────────────────────────────────────┬─────────────┘
│
ServiceLoader discovery │
▼
┌─────────────────────────────────────────────────────────────────────────┐
│ META-INF/services/org.apache.doris.protocol.ProtocolHandler │
│ │
│ org.apache.doris.mysql.MysqlProtocolHandler │
│ org.apache.doris.protocol.arrowflight.ArrowFlightProtocolHandler │
└────────────────────────────────────────────────────────────┬─────────────┘
│
instantiate + initialize │
▼
┌─────────────────────────────────────────────────────────────────────────┐
│ MysqlProtocolHandler (fe-protocol-mysql) │
│ │
│ - initialize(config): read port, ioThreads, backlog, etc │
│ - setAcceptor(callback): receive fe-core connection handling callback │
│ - start(): create XNIO server, start listening │
│ - on new connection: acceptor.accept(connection) │
└─────────────────────────────────────────────────────────────────────────┘

Configuration Flow

fe-core builds a ProtocolConfig and passes it into protocol handlers:

// QeService.javaprivateProtocolConfigbuildProtocolConfig() {
ProtocolConfigconfig = newProtocolConfig(mysqlPort, arrowFlightPort, scheduler);
// From Configconfig.set(KEY_MYSQL_IO_THREADS, Config.mysql_service_io_threads_num);
config.set(KEY_MYSQL_BACKLOG, Config.mysql_nio_backlog_num);
config.set(KEY_MYSQL_KEEP_ALIVE, Config.mysql_nio_enable_keep_alive);
// From FrontendOptionsconfig.set(KEY_MYSQL_BIND_IPV6, FrontendOptions.isBindIPV6());
// External thread poolconfig.set(KEY_MYSQL_TASK_EXECUTOR, ThreadPoolManager.newDaemon...());
returnconfig;
}

Configuration Key Mapping

fe-core parameter	ProtocolConfig key	Notes
`Config.mysql_service_io_threads_num`	`mysql.io.threads`	IO threads (default 4)
`Config.mysql_nio_backlog_num`	`mysql.backlog`	backlog size (default 1024)
`Config.mysql_nio_enable_keep_alive`	`mysql.keep.alive`	TCP keep-alive
`FrontendOptions.isBindIPV6()`	`mysql.bind.ipv6`	bind IPv6
`Config.max_mysql_service_task_threads_num`	`mysql.max.task.threads`	max task threads
`ThreadPoolManager` executor	`mysql.task.executor`	external thread pool

Design Principles

Protocol modules are independent
- Each protocol (MySQL, Arrow Flight) is a standalone Java module.
- Modules do not depend on each other.
Kernel is decoupled from protocols
- Kernel uses abstract SPI interfaces only.
- Protocol modules implement SPI and provide services.
Shared API can evolve
- SPI and shared API can expand for new protocols/features.
Protocol compatibility must not break
- MySQL protocol packet format, crypto, and handshake must remain compatible.
SPI extension mechanism
- New protocols are discovered via SPI; kernel does not hardcode implementations.

Migration Status

Classes already moved to protocol modules

Protocol module class	Original fe-core class	Notes
`o.a.d.protocol.mysql.MysqlCapability`	`o.a.d.mysql.MysqlCapability`	capability flags
`o.a.d.protocol.mysql.MysqlCommand`	`o.a.d.mysql.MysqlCommand`	command enum
`o.a.d.protocol.mysql.MysqlServerStatusFlag`	`o.a.d.mysql.MysqlServerStatusFlag`	server status
`o.a.d.protocol.mysql.MysqlColType`	`o.a.d.catalog.MysqlColType`	MySQL type codes
`o.a.d.protocol.mysql.MysqlPacket`	`o.a.d.mysql.MysqlPacket`	base packet
`o.a.d.protocol.mysql.MysqlHandshakePacket`	`o.a.d.mysql.MysqlHandshakePacket`	handshake
`o.a.d.protocol.mysql.MysqlAuthPacket`	`o.a.d.mysql.MysqlAuthPacket`	auth packet
`o.a.d.protocol.mysql.MysqlAuthSwitchPacket`	`o.a.d.mysql.MysqlAuthSwitchPacket`	auth switch
`o.a.d.protocol.mysql.MysqlOkPacket`	`o.a.d.mysql.MysqlOkPacket`	OK packet
`o.a.d.protocol.mysql.MysqlErrPacket`	`o.a.d.mysql.MysqlErrPacket`	error packet
`o.a.d.protocol.mysql.MysqlEofPacket`	`o.a.d.mysql.MysqlEofPacket`	EOF packet
`o.a.d.protocol.mysql.MysqlClearTextPacket`	`o.a.d.mysql.MysqlClearTextPacket`	clear text
`o.a.d.protocol.mysql.MysqlSslPacket`	`o.a.d.mysql.MysqlSslPacket`	SSL request
`o.a.d.protocol.mysql.MysqlColDef`	`o.a.d.mysql.MysqlColDef`	column definition
`o.a.d.protocol.mysql.FieldInfo`	`o.a.d.mysql.FieldInfo`	field metadata
`o.a.d.protocol.mysql.MysqlSerializer`	`o.a.d.mysql.MysqlSerializer`	serializer
`o.a.d.protocol.mysql.MysqlProto`	`o.a.d.mysql.MysqlProto`	protocol utils
`o.a.d.protocol.mysql.MysqlPassword`	`o.a.d.mysql.MysqlPassword`	password crypto
`o.a.d.protocol.mysql.BytesChannel`	`o.a.d.mysql.BytesChannel`	channel interface
`o.a.d.protocol.mysql.SslEngineHelper`	`o.a.d.mysql.SslEngineHelper`	SSL utilities

Classes still in fe-core (kernel dependencies)

These remain in fe-core due to heavy coupling with kernel classes (Config,
ConnectContext, QueryState, Auth, etc):

Class	Dependencies	Reason
`MysqlChannel`	ConnectContext	connection context needed
`MysqlSslContext`	Config	SSL configuration
`MysqlProto.negotiate()`	ConnectContext, Env, Auth	auth/handshake logic
`MysqlSerializer.writeField()`	Type	result serialization with kernel types
`ReadListener`	ConnectContext, ConnectProcessor	query pipeline
`ProxyProtocolHandler`	kernel classes	proxy protocol support
`authenticate/`	privileges	auth logic
`privilege/`	privileges	permission checks

Why full decoupling is still hard

Connection lifecycle
- ConnectContext owns session state, variables, transaction state.
Auth and privilege checks
- MySQL handshake calls Env.getAuth() directly.
Query execution
- ConnectProcessor drives parsing, planning, execution.
Result serialization
- MysqlSerializer.writeField() maps kernel Type to MySQL types.

Current Decoupling Strategy

┌─────────────────────────────────────────────────────────────────────────┐
│ fe-protocol-mysql (decoupled) │
│ │
│ OK Network layer: XNIO server, acceptor │
│ OK Protocol definitions: packets, fields, command codes │
│ OK Base serialization and crypto │
│ OK Handshake/Auth packets, OK/Err/EOF packets │
└─────────────────────────────────────────────────────────────────────────┘
↑
acceptor callback
│
┌─────────────────────────────────────────────────────────────────────────┐
│ fe-core (still coupled) │
│ │
│ WARN Connection handling: MysqlChannel, ReadListener │
│ WARN Protocol negotiation: MysqlProto.negotiate() │
│ WARN Query processing: ConnectProcessor │
│ WARN Result serialization: MysqlSerializer.writeField(Type) │
│ WARN Auth/privilege: authenticate/, privilege/ │
└─────────────────────────────────────────────────────────────────────────┘

Module Layout

fe/
├── fe-common/ # shared utilities
├── fe-protocol/ # protocol parent
│ ├── fe-protocol-api/ # SPI interfaces
│ │ └── org.apache.doris.protocol/
│ │ ├── ProtocolHandler.java
│ │ ├── ProtocolConfig.java
│ │ ├── ProtocolContext.java
│ │ ├── ProtocolLoader.java
│ │ ├── ProtocolException.java
│ │ └── AuthenticationResult.java
│ │
│ ├── fe-protocol-mysql/ # MySQL protocol implementation
│ │ └── org.apache.doris.protocol.mysql/
│ │ ├── MysqlProtocolHandler.java
│ │ ├── MysqlProto.java
│ │ ├── command/
│ │ │ └── MysqlCommand.java
│ │ └── channel/
│ │ └── MysqlChannel.java
│ │
│ └── fe-protocol-arrowflight/ # Arrow Flight SQL implementation
│ └── org.apache.doris.protocol.arrowflight/
│ ├── ArrowFlightProtocolHandler.java
│ └── FlightSqlContext.java
│
└── fe-core/ # kernel
└── org.apache.doris.qe/
└── QeService.java # loads protocols via SPI

Module Dependencies

 ┌─────────────────────────────────────┐
│ fe-common │
│ (shared utilities) │
└─────────────────────────────────────┘
▲
┌───────────────────────┼───────────────────────┐
│ │ │
│ │ │
┌───────────┴───────────┐ │ ┌───────────┴───────────┐
│ fe-protocol-api │ │ │ fe-protocol-api │
│ (SPI) │ │ │ (SPI) │
└───────────────────────┘ │ └───────────────────────┘
▲ │ ▲
│ │ │
┌───────────┴───────────┐ ┌────────┴────────┐ ┌───────────┴───────────┐
│ fe-protocol-mysql │ │ fe-core │ │ fe-protocol-arrowflight│
│ (MySQL impl) │◄──│ (kernel) │──►│ (Arrow Flight impl) │
└───────────────────────┘ └─────────────────┘ └───────────────────────┘

SPI Interfaces

ProtocolHandler

publicinterfaceProtocolHandler {
StringgetProtocolName();
StringgetProtocolVersion();
voidinitialize(ProtocolConfigconfig) throwsProtocolException;
voidsetAcceptor(Consumer<Object> acceptor);
booleanstart();
voidstop();
booleanisRunning();
intgetPort();
booleanisEnabled(ProtocolConfigconfig);
intgetPriority();
}

ProtocolContext

publicinterfaceProtocolContext {
StringgetProtocolName();
longgetConnectionId();
StringgetRemoteIP();
StringgetUser();
StringgetDatabase();
voidsetDatabase(Stringdatabase);
booleanisAuthenticated();
booleanisKilled();
voidsetKilled();
voidcleanup();
<T> TgetChannel();
}

Kernel Usage Example

QeService loads protocol handlers via SPI and registers acceptors:

publicclassQeService {
publicQeService(intmysqlPort, intarrowFlightPort, ConnectSchedulerscheduler) {
ProtocolConfigconfig = buildProtocolConfig();
List<ProtocolHandler> handlers = ProtocolLoader.loadConfiguredProtocols(config);
for (ProtocolHandlerhandler : handlers) {
if ("mysql".equalsIgnoreCase(handler.getProtocolName())) {
handler.setAcceptor(this::handleMysqlConnection);
} elseif ("arrowflight".equalsIgnoreCase(handler.getProtocolName())) {
handler.setAcceptor(this::handleArrowFlightConnection);
}
protocolHandlers.add(handler);
}
}
}

Add a New Protocol

Create a new module:

mkdir -p fe/fe-protocol/fe-protocol-newprotocol/src/main/java/org/apache/doris/protocol/newprotocol
mkdir -p fe/fe-protocol/fe-protocol-newprotocol/src/main/resources/META-INF/services

Implement ProtocolHandler:

packageorg.apache.doris.protocol.newprotocol;
publicclassNewProtocolHandlerimplementsProtocolHandler {
@OverridepublicStringgetProtocolName() { return"newprotocol"; }
@Overridepublicvoidinitialize(ProtocolConfigconfig) throwsProtocolException { }
@Overridepublicbooleanstart() { returntrue; }
}

Register SPI service:

META-INF/services/org.apache.doris.protocol.ProtocolHandler

org.apache.doris.protocol.newprotocol.NewProtocolHandler

Add module pom and dependency on fe-protocol-api.
Add dependency in fe-core to load the new protocol.

Arrow Flight: Current Coupling

Arrow Flight SQL is still tightly coupled with fe-core:

QeService constructs Arrow Flight objects (FlightTokenManagerImpl,
DorisFlightSqlProducer, FlightBearerTokenAuthenticator) and injects them
into ProtocolConfig, which mixes protocol-specific runtime objects into the
kernel config path.
DorisFlightSqlService is still launched by QeService when SPI handler is not
present (legacy fallback).
FlightSqlConnectProcessor and FlightSqlConnectContext still depend on
ConnectContext, ConnectScheduler, Auth, and execution pipeline classes.
ConnectScheduler owns FlightSqlConnectPoolMgr, used by session/token logic
and by connection limit enforcement.

Backward Compatibility

MySQL Protocol Compatibility

Packet formats unchanged
Crypto/SSL unchanged
Handshake flow unchanged
Command coverage unchanged

Migration Strategy

Keep existing org.apache.doris.mysql.* package as-is
Protocol modules can delegate to legacy implementation
Kernel calls protocols via SPI
Gradual migration without breaking clients

Configuration Parameters

Parameter	Description	Default
`query_port`	MySQL protocol port	9030
`arrow_flight_sql_port`	Arrow Flight SQL port	9090
`enable_ssl`	enable SSL	false
`max_connection_scheduler_threads_num`	max connections	4096

References

MySQL Protocol Documentation
Arrow Flight SQL Specification
Java ServiceLoader

TODO / Next Steps

The current protocol split is incomplete. Coupling remains high in user/session
management, configuration wiring, and connection pool management. The following
items are the prioritized next steps:

Separate user/session management from protocol handlers
- Introduce a kernel-facing AuthenticationService / SessionService SPI so
  protocol modules do not call Env.getAuth() or access user limits directly.
- Move user identity, session variables, and per-user limits into a protocol-
  neutral service. Protocols should only pass credentials and connection info.
Decouple configuration and parameter wiring
- Replace direct Config/FrontendOptions reads inside QeService with a
  dedicated ProtocolConfigFactory that builds protocol-scoped configs.
- Avoid injecting protocol-specific runtime objects (token managers, producers,
  executors) into ProtocolConfig. Instead, let protocol modules create and
  own these objects behind SPI boundaries, or supply them via dedicated SPI
  providers.
Extract connection pool management
- Move ConnectPoolMgr and FlightSqlConnectPoolMgr behind a unified
  ConnectionPoolService in fe-core.
- Protocol handlers should register/unregister connections through the SPI
  service rather than reaching into ConnectScheduler directly.
Define connection lifecycle SPI
- Standardize onConnect, onAuthenticate, onQuery, onClose hooks so
  MySQL and Arrow Flight share a consistent lifecycle, and kernel code owns
  the execution pipeline.
Finish Arrow Flight migration
- Move DorisFlightSqlService startup and token/session management into
  fe-protocol-arrowflight.
- Remove the legacy fallback path from QeService after migration.
Reduce kernel type leakage
- Introduce a protocol-neutral type mapping layer to reduce direct dependency
  on Type in serializers.
Add tests for SPI wiring
- Validate handler discovery, config mapping, and connection lifecycle for
  each protocol module.

hello-stephen · 2026-01-29T08:45:27Z

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR.

Please clearly describe your PR:

What problem was fixed (it's best to include specific error reporting information). How it was fixed.
Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
What features were added. Why was this function added?
Which code was refactored and why was this part of the code refactored?
Which functions were optimized and what is the difference before and after the optimization?

# Conflicts: # fe/fe-core/src/main/java/org/apache/doris/mysql/MysqlPassword.java # fe/fe-core/src/test/java/org/apache/doris/qe/ConnectContextTest.java

CalvinKirs · 2026-01-29T12:13:16Z

run buildall

CalvinKirs · 2026-01-29T12:21:24Z

run buildall

CalvinKirs · 2026-01-29T12:49:48Z

run buildall

CalvinKirs · 2026-01-30T01:56:29Z

run buildall

CalvinKirs · 2026-01-30T03:08:41Z

run buildall

When an insert task is canceled during execution, the shared `ctx` object may be set to null. Currently, after `command.runWithUpdateInfo(...)`, the code directly accesses `ctx.getState()`. If `ctx` is null or its state is null, this leads to a NullPointerException (NPE). ### Solution - Cache the `ctx` reference to a local variable to avoid race conditions. - Check both `ctx` and `ctx.getState()` for null before accessing. - If the task was canceled or the state is null, safely return without throwing an exception. - Maintain existing behavior: if the task completes with a non-OK state, still throw `JobException`. ### Impact - Prevents NPE when a task is canceled during execution. - Makes the insert task more robust in concurrent cancel scenarios.

hello-stephen · 2026-01-30T04:25:05Z

FE UT Coverage Report

Increment line coverage 10.37% (31/299) 🎉
Increment coverage report
Complete coverage report

CalvinKirs · 2026-01-30T07:00:22Z

run buildall

hello-stephen · 2026-01-30T08:28:28Z

FE UT Coverage Report

Increment line coverage 10.13% (31/306) 🎉
Increment coverage report
Complete coverage report

github-actions · 2026-08-13T00:19:23Z

We're closing this PR because it hasn't been updated in a while.
This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and feel free a maintainer to remove the Stale tag!

CalvinKirs added 4 commits January 27, 2026 16:49

MySQL OIDC
56095e6

Plugin(Protocol)
e486de0

arrow
f71de00

[feat](protocol)extract protocol definitions (part 1)
a0e7334

CalvinKirs requested a review from morningman as a code owner January 29, 2026 08:45

CalvinKirs added 4 commits January 29, 2026 17:09

[feat](protocol)extract protocol definitions (part 1)
df94cd7

[feat](protocol)extract protocol definitions (part 1)
c57f90c

Merge remote-tracking branch 'rich/master' into master-new-protocol
0503a86
# Conflicts: # fe/fe-core/src/main/java/org/apache/doris/mysql/MysqlPassword.java # fe/fe-core/src/test/java/org/apache/doris/qe/ConnectContextTest.java

[feat](protocol)extract protocol definitions (part 1)
d00b2cb

[feat](protocol)extract protocol definitions (part 1)
aca0c58

[feat](protocol)extract protocol definitions (part 1)
0d3c7e4

[feat](protocol)extract protocol definitions (part 1)
e8381d3

Merge branch 'master' into master-new-protocol
3bc60e9

CalvinKirs marked this pull request as draft February 2, 2026 06:41

morningman added the dev/4.1.x label Feb 8, 2026

CalvinKirs removed the dev/4.1.x label Feb 13, 2026

morningman mentioned this pull request Jul 15, 2026
[Tracking] Arrow Flight SQL and ADBC Catalog: issues and improvements #65615
Open
63 tasks

github-actionsBot added the Stale label Aug 13, 2026

CalvinKirs closed this Aug 13, 2026

Navigation Menu

Uh oh!

[feat](protocol)extract protocol definitions (part 1) - #60355

[feat](protocol)extract protocol definitions (part 1)#60355
CalvinKirs wants to merge 13 commits into
apache:masterfrom
CalvinKirs:master-new-protocol

CalvinKirs commented Jan 29, 2026•
edited
Loading

Uh oh!

hello-stephen commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 30, 2026

Uh oh!

CalvinKirs commented Jan 30, 2026

Uh oh!

hello-stephen commented Jan 30, 2026

Uh oh!

CalvinKirs commented Jan 30, 2026

Uh oh!

hello-stephen commented Jan 30, 2026

Uh oh!

github-actionsBot commented Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Uh oh!

Conversation

CalvinKirs commented Jan 29, 2026• edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What problem does this PR solve?

Overview

Core Design

SPI Loading Flow

Configuration Flow

Configuration Key Mapping

Design Principles

Migration Status

Classes already moved to protocol modules

Classes still in fe-core (kernel dependencies)

Why full decoupling is still hard

Current Decoupling Strategy

Module Layout

Module Dependencies

SPI Interfaces

ProtocolHandler

ProtocolContext

Kernel Usage Example

Add a New Protocol

Arrow Flight: Current Coupling

Backward Compatibility

MySQL Protocol Compatibility

Migration Strategy

Configuration Parameters

References

TODO / Next Steps

Uh oh!

hello-stephen commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 29, 2026

Uh oh!

CalvinKirs commented Jan 30, 2026

Uh oh!

CalvinKirs commented Jan 30, 2026

Uh oh!

hello-stephen commented Jan 30, 2026

FE UT Coverage Report

Uh oh!

CalvinKirs commented Jan 30, 2026

Uh oh!

hello-stephen commented Jan 30, 2026

FE UT Coverage Report

Uh oh!

github-actionsBot commented Aug 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

CalvinKirs commented Jan 29, 2026•
edited
Loading