Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

[PPL] init basic codebase structure #409

Conversation

penghuo
Copy link
Contributor

@penghuo penghuo commented Apr 6, 2020

Issue #, if available: #257

Purpose of the PR

1.PoC the idea proposed in #257.
2.Unblock the development of PPL and SQL migration.

Description of changes

  1. Add the basic codebase structure as proposed in Proposal for separating components into sub-modules #257. Three sub modules is added.
    1.1.plugin, Implement the Elasticsearch required Plugin framework.
    1.2.ppl, Implement the PPL specific logic.
    1.3.integ-test, integration test.

Notes

1.The source code is purely used to PoC. it will be changed later.
2.The build.gradle is not optimized. Pending on the move sql to sub-module.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@penghuo penghuo added the PPL label Apr 6, 2020
Copy link
Member

@zhongnansu zhongnansu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment on lines -19 to +18
// When upgrading to version after 7.3.0, must remove also
// project substitution from configurations.all (line 39)
es_version = System.getProperty("es.version", "7.6.1")
es_version = "7.6.1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nothing about this PR, but I wonder if another PR is needed to remove these deprecated in master as well. I can run some tests and help with that.

@penghuo penghuo merged commit 9f0d57a into opendistro-for-elasticsearch:feature/ppl Apr 6, 2020
@penghuo penghuo deleted the feature/ppl-framework branch September 17, 2020 16:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants