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.

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

adding support for array and dictionary expression parsing #106

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

lafar6502
Copy link

No description provided.

@JonathanMagnan JonathanMagnan self-assigned this Sep 2, 2020
@JonathanMagnan
Copy link
Member

Hello @lafar6502 ,

Thank you for the pull, I will ask my developer to review it

Best Regards,

Jon


Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework ExtensionsEntity Framework ClassicBulk OperationsDapper Plus

Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval FunctionSQL Eval Function

@ghost
Copy link

ghost commented Sep 3, 2020

Hi @lafar6502,

Can you please provide some details about the use case you've needed to instantiate array or dictionary as part of the query?

@JonathanMagnan
Copy link
Member

Hello @lafar6502 ,

We reviewed your pull, the code seems fine but as @meatGUY , we currently fail to see how you will really use it/if it should really be part of the library.

At this moment, it doesn't look very useful in LINQ statement and we don't want this library to become an Expression Evaluator library either which supports many different keywords.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants