Repository navigation
Update property defaults and deprecate old properties - #2495
Merged
Merged
Conversation
Added sections for adjusted, deprecated, and removed properties, as well as removed Java APIs in migration documentation.
…perty-changes - Add Maven 3.9.10 to minimum versions - Add cds.security.authentication.mode default change - Add cds.query.deepEntityReadonly default change - Add explanations to all property default rows - Fix typo: enforceTransational → enforceTransactional - Fix cds.multitenancy casing and use PT20M/PT60M notation - Fix cds.errors.combined removal description
rjayasinghe
force-pushed
the
StefanHenke-patch-3
branch
from
June 2, 2026 11:34
b552a2d to
8b63344
Compare
rjayasinghe
previously approved these changes
Jun 2, 2026
Contributor
|
@renejeglinsky @smahati this is approved now. Can you please go ahead with this change? |
agoerler
reviewed
Jun 3, 2026
agoerler
reviewed
Jun 3, 2026
agoerler
reviewed
Jun 5, 2026
…ert) Cross-checked the property tables against rel-4.9.x → main on cds-services (HEAD 50e3804088, includes the revert of #6812 in cds-services PR #6954). Adjusted Property Defaults: - Drop cds.odataV2.batch.maxRequests row (change reverted in cds-services PR #6954; both V2 and V4 are back to -1). - Drop cds.odatav2/v4.searchMode rows (the property is removed entirely, not defaulted; moved to "Removed Properties" with a behavior note). - Fix casing: cds.multitenancy.… → cds.multiTenancy.… (matches Java field). Deprecated Properties: - Convert bullet list to a table for consistency with the surrounding sections. - Add cds.dashboard.* — the Dashboard class is now @deprecated(forRemoval). Removed Properties: - Strip stray spaces in cds.sql.hana.optimizationMode. - Add the two removed cds.odata{V2,V4}.searchMode entries with a note that the runtime now behaves as if pass-through were set. - Add cds.multiTenancy.serviceManager.acceptInstancesWithoutTenant, cds.multiTenancy.serviceManager.ignoreDuplicateTenantInstances, and cds.sql.collate (silent breaking removals on main).
Contributor
|
@StefanHenke I checked with Claude Code and manually 👀 and I think we have everything in that we need. |
Contributor
|
will take care of the conflicts now |
# Conflicts: # java/migration.md
Contributor
Author
Looking at the list of deleted properties, I´m not sure if we really have recipes for all of them. Some of them are very unlikely to be used by anybody. But others maybe. |
Contributor
|
No, we don't have migrations for all of them. But let's merge this and work on the migrations. Once we have them we can update the table |
MattSchur
reviewed
Jun 18, 2026
agoerler
previously approved these changes
Jun 19, 2026
agoerler
enabled auto-merge
June 19, 2026 11:32
agoerler
previously approved these changes
Jun 19, 2026
agoerler
reviewed
Jun 19, 2026
Co-authored-by: René Jeglinsky <rene.jeglinsky@sap.com>
renejeglinsky
approved these changes
Jun 19, 2026
Merged
sjvans
pushed a commit
to BobdenOs/capire
that referenced
this pull request
Jun 24, 2026
Those were added by capire#2495 but were not detected by the link checks that we removed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Covers all changes for properties that we know of today. For all other properties under discussion, separate BLIs exist.