Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CBL-6737: Merge branch 'release/3.2' to the master branch #2224

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

jianminzhao
Copy link
Contributor

No description provided.

velicuvlad and others added 12 commits January 7, 2025 13:24
Change-Id: Ibc0efec8cff9eea29d1fbbfb087e931f92e09fa1
…essage (#2178)

We enhanced the error by adding to the original message, "This table is referenced by an array index, which may have been deleted."

* We also enhanced the error message of FTS index in the similar situation.
#2214)

* CBL-6651: createIndex doesn't support creating partial index with N1QL language

Extended IndexSpec to allow the where clause when the query langugage is N1QL. We put it in IndexSpec::options. In this commit, we only do it for ValueIndex and FTSIndex.

With JSON query language, the where clause can be either in the new option or in IndexSpec::expression together with the what clause. With N1QL query language, the where clause has to be in the field of IndexSpec, and IndexSpec::expression contains only the what clause.

This commit is to get ready for C4IndexOptions
Added field "where" to C4IndexOptions and implemented it.
The removal of restriction of "LIMIT > 0" matches that in 4.0 line.
* CBL-6534: Fix expiration column upgrade (#2195)

Co-authored-by: jianminzhao <[email protected]>
#2220)

* CBL-6671: Options of FTS partial index is not fully encoded in the SQLite table when it is registered

1. Bumped Schema version.
2. In the new version, added a column, whereClause, to table indexes.
3. Upgrade the tables in previous version to include the new column
4. IndexSpec::whereClause is registered in table "indexes" by the new column.
5. c4index_getOptions will have "where" passed in with C4IndexOptions.
6. Tests of the above.
@jianminzhao jianminzhao changed the title Cbl 6737 CBL-6737: Merge branch 'release/3.2' to the master branch Feb 6, 2025
@cbl-bot
Copy link

cbl-bot commented Feb 7, 2025

Code Coverage Results:

Type Percentage
branches 66.41
functions 78.63
instantiations 70.74
lines 77.73
regions 73.48

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants