Skip to content

Commit

Permalink
Restoring skipped pushdown tests (#125) (#127)
Browse files Browse the repository at this point in the history
* Restoring skipped pushdown tests



* Fix backport workflow



---------


(cherry picked from commit 038c5d5)

Signed-off-by: Harsha Vamsi Kalluri <harshavamsi096@gmail.com>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
  • Loading branch information
1 parent f4198b5 commit 0876b1e
Show file tree
Hide file tree
Showing 5 changed files with 12 additions and 13 deletions.
12 changes: 11 additions & 1 deletion .github/workflows/backport.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,17 @@ on:

jobs:
backport:
if: ${{ contains(github.event.label.name, 'backport') }}
# Only react to merged PRs for security reasons.
# See https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request_target.
if: >
github.event.pull_request.merged
&& (
github.event.action == 'closed'
|| (
github.event.action == 'labeled'
&& contains(github.event.label.name, 'backport')
)
)
runs-on: ubuntu-latest
permissions:
contents: write
Expand Down
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
### Deprecated
### Removed
### Fixed
- Restored skipped push down tests ([125](https://github.com/opensearch-project/opensearch-hadoop/pull/125))
### Security
### Dependencies
- Bumps `com.google.guava:guava` from 16.0.1 to 23.0
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1010,8 +1010,6 @@ class AbstractScalaOpenSearchScalaSparkSQL(prefix: String, readMetadata: jl.Bool
}

@Test
@Ignore
// @AwaitsFix(bugUrl = "https://github.com/opensearch-project/opensearch-hadoop/issues/41")
def testDataSourcePushDown09StartsWith() {
val df = opensearchDataSource("pd_starts_with")
var filter = df.filter(df("airport").startsWith("O"))
Expand All @@ -1032,8 +1030,6 @@ class AbstractScalaOpenSearchScalaSparkSQL(prefix: String, readMetadata: jl.Bool
}

@Test
@Ignore
// @AwaitsFix(bugUrl = "https://github.com/opensearch-project/opensearch-hadoop/issues/41")
def testDataSourcePushDown10EndsWith() {
val df = opensearchDataSource("pd_ends_with")
var filter = df.filter(df("airport").endsWith("O"))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1066,8 +1066,6 @@ class AbstractScalaOpenSearchScalaSparkSQL(prefix: String, readMetadata: jl.Bool
}

@Test
@Ignore
// @AwaitsFix(bugUrl = "https://github.com/opensearch-project/opensearch-hadoop/issues/41")
def testDataSourcePushDown09StartsWith() {
val df = opensearchDataSource("pd_starts_with")
var filter = df.filter(df("airport").startsWith("O"))
Expand All @@ -1088,8 +1086,6 @@ class AbstractScalaOpenSearchScalaSparkSQL(prefix: String, readMetadata: jl.Bool
}

@Test
@Ignore
// @AwaitsFix(bugUrl = "https://github.com/opensearch-project/opensearch-hadoop/issues/41")
def testDataSourcePushDown10EndsWith() {
val df = opensearchDataSource("pd_ends_with")
var filter = df.filter(df("airport").endsWith("O"))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1068,8 +1068,6 @@ class AbstractScalaOpenSearchScalaSparkSQL(prefix: String, readMetadata: jl.Bool
}

@Test
@Ignore
// @AwaitsFix(bugUrl = "https://github.com/opensearch-project/opensearch-hadoop/issues/41")
def testDataSourcePushDown09StartsWith() {
val df = opensearchDataSource("pd_starts_with")
var filter = df.filter(df("airport").startsWith("O"))
Expand All @@ -1090,8 +1088,6 @@ class AbstractScalaOpenSearchScalaSparkSQL(prefix: String, readMetadata: jl.Bool
}

@Test
@Ignore
// @AwaitsFix(bugUrl = "https://github.com/opensearch-project/opensearch-hadoop/issues/41")
def testDataSourcePushDown10EndsWith() {
val df = opensearchDataSource("pd_ends_with")
var filter = df.filter(df("airport").endsWith("O"))
Expand Down

0 comments on commit 0876b1e

Please sign in to comment.