-
Notifications
You must be signed in to change notification settings - Fork 60
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.
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
Snowpark backend #1133
Merged
Merged
Snowpark backend #1133
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Generate empty files for ELM modules in the IR
* Add initial implementation of a 'mapping' context This context element has information about type definitions. * Code review suggestions
Add support to the literals using the snowpark function lit for boolean, string, char, float and Int. Add test for lit literals.
* Adds basic generation of record wrappers * Fix missing `extends` generation
Also adds local variables for records representing tables. Related to finos#20
Support List.member function into Snowpark using the "in" function
…s#48) * Add support for Snowpark generation of basic PatternMatch cases The following cases are supported: - Cases with literal values - Cases with constructors of union types without parameters
* Add support for some list functions Adds basic support for: - `List.map` with lambda returning record - `List.map` with lambda returning a single value - `List.filter` with lambda predicate - `List.sum` with `List.map` as its collection * Fixes for Code review comments
Generate a basic support for the add, subtract, multiply and divide operators in Snowpark
Support the comparison operators for snowpark ===, =!=, <, >, <=, >=
* Add support for a couple of elements - Adds basic support for case/of with Union types with params ``` case x of First a b c -> a Second y -> y ``` - Adds support for Maybe case/of patterns ``` case x of Just w -> w Nothing -> 19 ``` - Adds basic support for conversion of `Maybe.Just` and `Maybe.Nothing` - Adds a new Scala literal for `null` * Code review changes
Add support for the and (&&) or (||) and not (!) operators in snowpark.
Add support for the floor and module functions in Snowpark
* Add support `let` and `Maybe` functions Adds support for: - Basic `let` without function definitions - `Maybe.map` - `Maybe.withDefault` * Fix Nothing comparisons Also fixes tagging of fields with aliases.
Add support to access field of a record in the way .field
- Fix creation of union type instance with parameters - Fix generation of parameter names with Scala reserved words - Fix calls to use curry-like syntax in scala
* Add generation of wrappers for keeping record references Now we don't remove parameters representing records from functions . This fixes problems where the a column may be lead to ambiguities. * Code review suggestions
Changes to allow the conversion of must of the subset for Unsecured.elm. Adds basic unpacking of JSON results. Fixes finos#75
Adds support for converting tuple creation: let t = (1,2,3) And matching: case t of (a,b,c) -> c
…inos#88) * Add support for the groupBy and aggregate from Morphir.SDK Add support to the pattern of groupBy and Aggregate from the Moprhir.SDK in Snowpark * Support multiple variables in the lambda's call for the SDK Aggregate function Support multiple variable in the lambda's call for the SDK aggregate function * Support the alias function from the column in order to generate a custom column name * Refactoring Aggregate mapping * Fix alias generation and refactoring constants and map functions file
Fix alias name when is added in the aggregate function for the column
Fix passing lambda functions as arguments.
* Changes to improve compilation for `Calculations.elm` Adds support for: - Different set of function mappings when converting a function identified as Scala-only - Convert empty DataFrame (still pending issue with session object) - Adds several simple mappings for `Calculations.elm`: `List.maximum` , `max`, `min` * Code review fixes and fix List.sum with null values
* Add support for the Aggregate when returns a record Support the return type as a record in the Aggregate function. * Support records and handle error asign in the record field
* First commit for decorations * Add support for `inlining`
First support for the join feature in elm. Mapping the innerJoin function from Elm
* Add issue generation report - Generates `GenerationReport.elm` - Adds propagation of issue list - Changes uses of `Value` for `TypedValue` * Code review suggestions
* In-progress documentation and fixes/coverage * Add minor updates * Add missing elements * Add latest changes * Split initial documents * Code review comments
- Fixes some pattern matching scenarios - Adds pattern matching test - Adds function mapping tests - Adds support for some function scenarios
Adds command line options for the `Snowpark` backend to the CLI2 project.
- Add Test for the join feature.
* Adding and fixing documentation for Snowpark backend
- Remove unused code - Fix Boolean expression generation in pattern matching code
* Add documentation for the join feature * Fixing documentation
Fixes: - `List.filter` with a local function reference - Binary operation generated as part of a projection
* Code review suggestion Changes string literal to "throwing an exception" when processing a not supported element. * Code review suggestion Add support for aggregate errors
17 tasks
stephengoldbaum
approved these changes
Jan 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds a new backend for generating Scala code that uses Snowpark .
Terms