Skip to content

SQL: Add project routing support to JDBC#138756

Merged
luigidellaquila merged 8 commits intoelastic:mainfrom
luigidellaquila:sql/jdbc_project_routing
Dec 3, 2025
Merged

SQL: Add project routing support to JDBC#138756
luigidellaquila merged 8 commits intoelastic:mainfrom
luigidellaquila:sql/jdbc_project_routing

Conversation

@luigidellaquila
Copy link
Copy Markdown
Member

Adding project routing support to JDBC.

@luigidellaquila luigidellaquila marked this pull request as ready for review December 3, 2025 08:45
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 3, 2025
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @luigidellaquila, I've created a changelog YAML for you.

public static final String ALLOW_PARTIAL_SEARCH_RESULTS_DEFAULT = "false";

// CPS project routing
public static final String PROJECT_ROUTING = "project.routing";
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For my education, jdbc does not allow _ in property names?
I am asking as it is project_routing everywhere else.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we have limitations in this sense.
I'm using dots for consistency, as all the other options use dots as well, also for well known parameters (eg. allow.partial.search.results)

@luigidellaquila luigidellaquila enabled auto-merge (squash) December 3, 2025 09:13
@luigidellaquila luigidellaquila enabled auto-merge (squash) December 3, 2025 10:32
@luigidellaquila luigidellaquila merged commit 7fe1d31 into elastic:main Dec 3, 2025
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/SQL SQL querying >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants