Skip to content

[BACKPORT][SPARK-18952] Regex strings not properly escaped in codegen for aggregations - #16518

Closed
brkyvz wants to merge 1 commit into
apache:branch-2.1from
brkyvz:reg-break-2.1
Closed

[BACKPORT][SPARK-18952] Regex strings not properly escaped in codegen for aggregations#16518
brkyvz wants to merge 1 commit into
apache:branch-2.1from
brkyvz:reg-break-2.1

Conversation

@brkyvz

@brkyvzbrkyvz commented Jan 9, 2017

Copy link
Copy Markdown
Contributor

What changes were proposed in this pull request?

Backport for #16361 to 2.1 branch.

How was this patch tested?

Unit tests

@brkyvz
brkyvz changed the base branch from master to branch-2.1January 9, 2017 20:01
@brkyvz
brkyvz changed the base branch from branch-2.1 to masterJanuary 9, 2017 20:02
@SparkQA

SparkQA commented Jan 9, 2017

Copy link
Copy Markdown

Test build #71089 has finished for PR 16518 at commit c2de5ee.

  • This patch fails to build.
  • This patch merges cleanly.
  • This patch adds no public classes.

@brkyvz
Save
6a05386
Fixed
use unnamed version
@brkyvz
brkyvz changed the base branch from master to branch-2.1January 9, 2017 20:10
@SparkQA

SparkQA commented Jan 9, 2017

Copy link
Copy Markdown

Test build #71090 has finished for PR 16518 at commit 6a05386.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@JoshRosen

JoshRosen commented Jan 9, 2017

Copy link
Copy Markdown
Contributor

Merging to branch-2.1. Thanks!

asfgit pushed a commit that referenced this pull request Jan 9, 2017
@brkyvz@JoshRosen
[SPARK-18952][BACKPORT] Regex strings not properly escaped in codegen…
3b6ac32
… for aggregations
## What changes were proposed in this pull request?
Backport for #16361 to 2.1 branch.
## How was this patch tested?
Unit tests
Author: Burak Yavuz <brkyvz@gmail.com>
Closes#16518 from brkyvz/reg-break-2.1.
@brkyvz

brkyvz commented Jan 10, 2017

Copy link
Copy Markdown
ContributorAuthor

Closing this since it was backported

@brkyvzbrkyvz closed this Jan 10, 2017
@brkyvz
brkyvz deleted the reg-break-2.1 branch February 3, 2019 20:58
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@brkyvz@SparkQA@JoshRosen