Skip to content

Conversation

@russcam
Copy link
Contributor

@russcam russcam commented Oct 18, 2018

This Pull Request is to merge the first set of 6.4 features into 6.x, for a first 6.4.0 client release.

Mpdreamz and others added 30 commits September 20, 2018 10:56
…ild.bat was never called which pulls down the jetBRains self host profiler SKD
…to us using IgnoreUnavailable which was basically ignored in previous version
…ertionScope which loses stack trace of the failed assertions (by design)
…n the jetBrains Rider/Resharper test runner that removes tests when you open the test file https://youtrack.jetbrains.com/issue/RIDER-19912 (#3418)
…s prevents unexpected results when running tests from the IDE which is the primary usecase for
* Add QuerySql API methods and tests
* Add ClearSqlCursor API methods and tests
* Add TranslateSql API methods and tests
It will no longer allow false and will be removed on the server in 8.x, and in the client in 7.x
This commit refactors the way we do analysis tests by making it easier for test classes to do full blown endpoint testing

(cherry picked from commit d6f1ae5)

* continuing to port analysis over to more isolated chunks

(cherry picked from commit f5f0c437871589b1fb90b6c4c6f09f0dfc296d7e)

* Ported normalizers over to new test format

(cherry picked from commit c74ed51e2c30804ffc1d50f95a17893a93bfa6ea)

* clean up namespaces

(cherry picked from commit f2da9f51b43b188cc1b2d09f616fbf87ca268344)

* All analysis unit/integ tests pass again

(cherry picked from commit 7ecbee5435df02810ede7f07985e7bb13f66b6f3)

* abstracted analysis base classes even further to all share a base which implements the bulk of the setup and tests

(cherry picked from commit 8a6e99493a4174a87cc8680609afd0c482cf10d7)
This commit adds support for weighted average aggregation
(cherry picked from commit 5eb0d78)
Mpdreamz and others added 27 commits October 17, 2018 15:21
* add analysis-nori plugin to writable cluster and added nori_tokenizer
* add nori_part_of_speech token filter
* add nori analyzer
* add Nori() to AnalyzeTokenDescriptor, CodeStandard tests caught this
This commit adds support for the Rollup APIs:

- CreateRollupJob
- GetRollupJob
- DeleteRollupJob
- StartRollupJob
- StopRollupJob
- GetRollupCapabilities
- RollupSearch
This commit re-runs the API generation against the 6.4 branch.
Ignore the new nodes.reload_secure_settings.json endpoint for now
This commit fixes XML comments, corrects file names and sets the correct
section header level for generated documentation.
This commit disposes the CancellationTokenSource when diposing the BulkAllObservable<T>.
Since the CancellationTokenSource could be linked to a passed CancellationToken
that sets a timeout, not disposing it can lead to flooding the timer queue, affecting performance.
This commit fixes the integration tests:

Name properties for analysis tests need to be unique
FieldAlias tests need the field to exist in the index.
Will allow the list tasks response to be written out in the event of a failed assertion
With the refactoring of test projects, the link to the original source files
in documentation also require updating
@russcam russcam requested a review from codebrain October 18, 2018 06:03
@russcam russcam merged commit cfa1af6 into 6.x Oct 18, 2018
@Mpdreamz Mpdreamz deleted the 6.4 branch November 1, 2018 09:01
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.

3 participants