Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use attachValues in SQL #11259

Merged
merged 7 commits into from
Apr 5, 2020
Merged

Use attachValues in SQL #11259

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
064f88c
foo
Mar 29, 2020
348e746
replace addValues with attachValues in SQL
Mar 29, 2020
4dd5391
fix comments
Mar 29, 2020
fb15f8a
fix BeamCalcRel
Apr 1, 2020
7f806d2
foo
Apr 2, 2020
82973bd
fix failures and add pipeline option to control
Apr 2, 2020
0155354
fix return
Apr 2, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
foo
  • Loading branch information
Reuven Lax committed Apr 2, 2020
commit 7f806d2c610c1ae9a51d90dbaa3059bbeca01217
1 change: 0 additions & 1 deletion ...asql/src/test/java/org/apache/beam/sdk/extensions/sql/zetasql/ZetaSQLDialectSpecTest.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4061,7 +4061,6 @@ public void testAVGWithLongInput() {
"AVG(LONG) is not supported. You might want to use AVG(CAST(expression AS DOUBLE).");
zetaSQLQueryPlanner.convertToBeamRel(sql);
}

@Test
public void testReverseString() {
String sql = "SELECT REVERSE('abc');";
Expand Down