* Add pit for join queries (#2703)
* Add search after for join
Signed-off-by: Rupal Mahajan <[email protected]>
* Enable search after by default
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit
Signed-off-by: Rupal Mahajan <[email protected]>
* nit
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix tests
Signed-off-by: Rupal Mahajan <[email protected]>
* ignore joinWithGeoIntersectNL
Signed-off-by: Rupal Mahajan <[email protected]>
* Rerun CI with scroll
Signed-off-by: Rupal Mahajan <[email protected]>
* Remove unused code and retrigger CI with search_after true
Signed-off-by: Rupal Mahajan <[email protected]>
* Address comments
Signed-off-by: Rupal Mahajan <[email protected]>
* Remove unused code change
Signed-off-by: Rupal Mahajan <[email protected]>
* Update pit keep alive time with SQL_CURSOR_KEEP_ALIVE
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix scroll condition
Signed-off-by: Rupal Mahajan <[email protected]>
* nit
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit before query execution
Signed-off-by: Rupal Mahajan <[email protected]>
* nit
Signed-off-by: Rupal Mahajan <[email protected]>
* Move pit from join request builder to executor
Signed-off-by: Rupal Mahajan <[email protected]>
* Remove unused methods
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit in parent class's run()
Signed-off-by: Rupal Mahajan <[email protected]>
* Add comment for fetching subsequent result in NestedLoopsElasticExecutor
Signed-off-by: Rupal Mahajan <[email protected]>
* Update comment
Signed-off-by: Rupal Mahajan <[email protected]>
* Add javadoc for pit handler
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit interface
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit handler unit test
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix failed unit test CI
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix spotless error
Signed-off-by: Rupal Mahajan <[email protected]>
* Rename pit class and add logs
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix pit delete unit test
Signed-off-by: Rupal Mahajan <[email protected]>
---------
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit for multi query (#2753)
* Add search after for join
Signed-off-by: Rupal Mahajan <[email protected]>
* Enable search after by default
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit
Signed-off-by: Rupal Mahajan <[email protected]>
* nit
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix tests
Signed-off-by: Rupal Mahajan <[email protected]>
* ignore joinWithGeoIntersectNL
Signed-off-by: Rupal Mahajan <[email protected]>
* Rerun CI with scroll
Signed-off-by: Rupal Mahajan <[email protected]>
* draft
Signed-off-by: Rupal Mahajan <[email protected]>
* Remove unused code and retrigger CI with search_after true
Signed-off-by: Rupal Mahajan <[email protected]>
* Address comments
Signed-off-by: Rupal Mahajan <[email protected]>
* Remove unused code change
Signed-off-by: Rupal Mahajan <[email protected]>
* Update pit keep alive time with SQL_CURSOR_KEEP_ALIVE
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix scroll condition
Signed-off-by: Rupal Mahajan <[email protected]>
* nit
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit before query execution
Signed-off-by: Rupal Mahajan <[email protected]>
* Refactor get response with pit method
Signed-off-by: Rupal Mahajan <[email protected]>
* Update remaining scroll search calls
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix integ test failures
Signed-off-by: Rupal Mahajan <[email protected]>
* nit
Signed-off-by: Rupal Mahajan <[email protected]>
* Move pit from join request builder to executor
Signed-off-by: Rupal Mahajan <[email protected]>
* Remove unused methods
Signed-off-by: Rupal Mahajan <[email protected]>
* Move pit from request to executor
Signed-off-by: Rupal Mahajan <[email protected]>
* Fix pit.delete call missed while merge
Signed-off-by: Rupal Mahajan <[email protected]>
* Move getResponseWithHits method to util class
Signed-off-by: Rupal Mahajan <[email protected]>
* add try catch for create delete pit in minus executor
Signed-off-by: Rupal Mahajan <[email protected]>
* move all common fields to ElasticHitsExecutor
Signed-off-by: Rupal Mahajan <[email protected]>
* add javadoc for ElasticHitsExecutor
Signed-off-by: Rupal Mahajan <[email protected]>
* Add missing javadoc
Signed-off-by: Rupal Mahajan <[email protected]>
* Forcing an empty commit as last commit is stuck processing updates
Signed-off-by: Rupal Mahajan <[email protected]>
---------
Signed-off-by: Rupal Mahajan <[email protected]>
* Add pit to default cursor
Signed-off-by: Rupal Mahajan <[email protected]>
* Run CI without pit unit test
Signed-off-by: Rupal Mahajan <[email protected]>
* Rerun CI without pit unit test
Signed-off-by: Rupal Mahajan <[email protected]>
* FIx unit tests for PIT changes
Signed-off-by: Manasvini B S <[email protected]>
* Addressed comments
Signed-off-by: Manasvini B S <[email protected]>
---------
Signed-off-by: Rupal Mahajan <[email protected]>
Signed-off-by: Manasvini B S <[email protected]>
Co-authored-by: Rupal Mahajan <[email protected]>
(cherry picked from commit 69853fe)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>