Draft
Conversation
Dltmd202
force-pushed
the
topic/Dltmd202/issue-3311
branch
5 times, most recently
from
40bb5df to
e22ed1a
Compare
Dltmd202
force-pushed
the
topic/Dltmd202/issue-3311
branch
2 times, most recently
from
bb9b5e8 to
d49dd31
Compare
…ition 8.2 (redis#3334) * bitop new operations * polish * update new API signatures in NodeSelectionCommands
* Migrate publishing to Maven Central Portal * Fix SNAPSHOT version format As required by https://central.sonatype.org/publish/publish-portal-snapshots/
* DOC-4758 added landing page examples * DOC-4758 added Path page examples * DOC-4758 work around arrAppend issue * DOC-4758 applied formatting
* Add support for 8.2 stream commands - Add support for XDELEX and XACKDEL - Extend XADD and XTRIM to support trimming policy * Add @SInCE tags * Update NodeSelection API * Add StreamEntryDeletionResult.UNKNOWN to avoid throwing exceptions * Clean up RedisCommandBuilder - Remove redundant code blocks - Use CommandKeyword for IDS - Add unit tests * Fix broken test
* test: remove redundant assertions and clarify time-related values with constants * test: remove redundant assertions and clarify time-related values with constants * test: remove redundant assertions and clarify time-related values with constants
* RediSearch support (part 1 of 3) (redis#3325) * Revert "Revert "Add support for FT.CREATE redis#2717 (redis#3150)" (redis#3160)" (redis#3161) This reverts commit 319e315. * Add support for FT.DROPINDEX redis#2722 (redis#3164) * Add support for FT.DROPINDEX redis#2722 * Polishing * FT.SEARCH added Vibe code the FT.SEARCH command Pulled latest from the integration branch Fixed a lot of the hallucinations RESP2 parser improvements JSON Indexing tests Add some advanced use-cases Fixed all integration tests Fixed schema fields, added vector tests, but they are all failing Fixed vector search tests Improve coverage of integration and unit tests One unit test fails * API cleanup, added Kotlin implementation * Implement the FT.AGGREGATE command Fixing the integration tests for FT.AGGREGATE Polishing 1/3 Add more documentation ot the AggregateArgs * Implement the FT.CURSOR command Add Integration tests for the FT.CURSOR Add RESP2 tests for all aggragation integration tests * FT.AGGREGATE was never working in the first place, now it is working, but for RESP3 only * Fixing unit tests (part1) Fixing unit tests (part2) Fixing unit tests (part3) * Polishing (part 1) * RediSearch support (part 2 of 3) (redis#3360) Introducing the FT.ALTER Introducing the FT.ALIASADD, FT.ALIASUPDATE, FT.ALIASDEL Introducing the FT.TAGVALS * RediSearch support (part 3 of 3) (redis#3364) * Introducing the FT.SUGADD FT.SUGDEL FT.SUGGET and FT.SUGLEN * Introducing the FT.DICTADD, FT.DICTDEL, FT.DICTDUMP * Introducing the FT.SPELLCHECK * Introducing the FT.SPELLCHECK (polishing) * Introducing the FT.EXPLAIN, FT.SYNUPDATE, FT.SYNDUMP, FT._LIST * Introducing the FT.PROFILE Polishing 1 Polishing 1 * Polishing 2 * Removing the FT.PROFILE command entirely * Polishing 3 * Polishing 4 * RediSearch with Lettuce - final fixes (redis#3371) * Fix wrong aggregation arguments order Docs part (to squash later) Modified CreateArgs and SearchArgs Address review comments 1/many Addressed review comment by Igor, more polishing * Cluster tests * Extend wordlist * Extend wordlist 2 * Avoid NPE when empty result received * Avoid NPE when empty result received 2 * Disable cluster tests for 7.2 and 7.4
* Provide support for the SVS-VMANA index * Enable VAMANA tests for Redis 8.2+
* reuse-objectmapper-defaultjsonparser Refactor DefaultJsonParser to use a single static ObjectMapper instance instead of creating new instances for each method call, improving performance and memory usage. * Allow injection of custom ObjectMapper in DefaultJsonParser * Add JavaDoc --------- Co-authored-by: Tihomir Mateev <tihomir.mateev@gmail.com>
* Releasing 6.8.0.RELEASE * Expanding the evergrowing wordlist * Expanding the evergrowing wordlist 2
…re (redis#3328) (redis#3346) * chore: remove deprecated ClientOptions#cancelCommandsOnReconnectFailure - removed the option from ClientOptions and ClusterClientOptions and related builders - removed usages from integration tests and unit tests - removed usage from ReconnectionHandler and DefaultEndpoint Signed-off-by: The-East-Wind <hariatul1998@gmail.com> * Also remove the documentation on the cancelCommandsOnReconnectFailure option --------- Signed-off-by: The-East-Wind <hariatul1998@gmail.com> Co-authored-by: Tihomir Mateev <tihomir.mateev@gmail.com>
…dis#3328) (redis#3344) * chore: remove deprecated AbstractRedisClient#getDefaultTimeout - remove usages from RedisClient and replaced with timeout value from redisUri - fixed failing tests in RedisClientConnectIntegrationTests by asserting timeout to be equal to default timeout from RedisURI - refactored DefaultRedisClient test fixture to not set timeout Signed-off-by: The-East-Wind <hariatul1998@gmail.com> * chore: remove deprecated default timeout from AbstractRedisClient - refactor AtLeastOnce and AtMostOnce IntegrationTests to remove setting timeout on the client level and instead set it in the URI level - removed default timeout and usages which set it from AbstractRedisClient, RedisClient and RedisClusterClient Signed-off-by: The-East-Wind <hariatul1998@gmail.com> * docs: update relevant documentation to remove usage of RedisClient#setDefaultTimeout Signed-off-by: The-East-Wind <hariatul1998@gmail.com> --------- Signed-off-by: The-East-Wind <hariatul1998@gmail.com>
* Remove deprecated withPassword(String) method * Completely remove all deprecated logic in the RedisURI class --------- Co-authored-by: Tihomir Mateev <tihomir.mateev@gmail.com>
* Enable adaptive refresh by default redis#3249 * Added @SInCE * Fix integration test [incomplete] * Add also fix to issue 3250 * Change default to 5 seconds * Remove unnecessary new line
…edis#3279) * Using non-native transports with SocketOptions should cause an error * Add LettuceAssert.assertState Revert previous implement
* Bump kotlin.version from 1.7.21 to 1.9.25 Bumps `kotlin.version` from 1.7.21 to 1.9.25. Updates `org.jetbrains.kotlin:kotlin-bom` from 1.7.21 to 1.9.25 - [Release notes](https://github.com/JetBrains/kotlin/releases) - [Changelog](https://github.com/JetBrains/kotlin/blob/v1.9.25/ChangeLog.md) - [Commits](JetBrains/kotlin@v1.7.21...v1.9.25) Updates `org.jetbrains.kotlin:kotlin-maven-plugin` from 1.7.21 to 1.9.25 --- updated-dependencies: - dependency-name: org.jetbrains.kotlin:kotlin-bom dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: org.jetbrains.kotlin:kotlin-maven-plugin dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> * Minimum language and api level is now 1.4 --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Tihomir Mateev <tihomir.mateev@gmail.com>
…edis#3266) - Changed constructor to private to prevent instantiation
Nowadays, AI reviewing tools like CoPilot catch spellcheck issues better than pyspelling without a burden of maintaining wordlist.txt
a-TODO-rov
force-pushed
the
topic/Dltmd202/issue-3311
branch
from
d49dd31 to
a462f34
Compare
* Add getCodec method to StatefulRedisClusterConnectionImpl * feat: Added `StatefulConnection.getCodec()` method * Formatting fix and and some re-wording * One file was missed in the original change --------- Co-authored-by: Tihomir Mateev <tihomir.mateev@gmail.com>
…edis#3465) * CAE-534 - Add sticky node connection support for ftAggregate with cursor. * Implement random LB for RediSearch read commands. Change sticky cursor implementation to propagate more lightweight Cursor object, instead the whole AggregationReply. * Sync templates and API generator * Add ftCreate. Get connection by nodeId * Improvements * equalize reactive * Format * Fix failing test * Format * Code cleanup. Add FT.ALTER, FT.DROPINDEX (and DD), FT.SYNDUMP, FT.SYNUPDATE * Revert unintentional changes * Return override to dbSize * Account ReadFrom user preferences * Minor fixes from code review * Add readOnly RediSearch commands to default readOnly commands * Fix test * Fix test on lower env * Keyless API * Remove keys from advanced commands * Minor fixes * Fix by templates * Add test * Rename method * Fix tests * Polishing after code review
* docs : replace() (in JsonArray.java) javadoc rewrite * feat : implement JsonArray.swap() * Add tests --------- Co-authored-by: aleksandar.todorov <a_t_todorov@yahoo.com>
…ttern) - deprecated since 6.0
This reverts commit a0a3dfe.
…String pattern) - deprecated since 6.0
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
20 participants
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.
Make sure that:
mvn formatter:formattarget. Don’t submit any formatting related changes.