Code Monkey home page Code Monkey logo

Comments (10)

jeremiah-carlin avatar jeremiah-carlin commented on May 24, 2024

After some testing the CommandBase.FromScript call is truncating json ingestion mappings at a single quote which breaks the downstream validation. For now I've hacked in a fix by filtering ingestion mappings out of the schemaOutput and then allowing CreateMappingCommand to handle the ingestion mappings as normal.

from delta-kusto.

vplauzon avatar vplauzon commented on May 24, 2024

I believe it is related to mappings now being provided by default in the .show db script command.

It is being fixed in version 0.11. It should be available for download in a few hours...

from delta-kusto.

vplauzon avatar vplauzon commented on May 24, 2024

Fixed in https://github.com/microsoft/delta-kusto/releases/tag/0.11.0.115

from delta-kusto.

jeremiah-carlin avatar jeremiah-carlin commented on May 24, 2024

Awesome! Thanks for the quick response.

Testing out 0.11.0.115 I'm getting the following error when running any command through delta-kusto

❯ bin/0.11.0.115/delta-kusto

delta-kusto 0.11.0.115
Session:  a067593f-4c4c-4634-b672-867885d79ad6

Exception encountered:  System.InvalidOperationException ; Type delta_kusto.CommandLineOptions appears to be immutable, but no constructor found to accept values.
Stack trace:     at CommandLine.Core.InstanceBuilder.BuildImmutable[T](Type, Maybe`1, IEnumerable`1, IEnumerable`1, List`1)
   at CommandLine.Core.InstanceBuilder.<>c__DisplayClass1_0`1.<Build>b__5()
   at CommandLine.Core.InstanceBuilder.Build[T](Maybe`1, Func`3, IEnumerable`1, StringComparer, Boolean, CultureInfo, Boolean, Boolean, Boolean, IEnumerable`1)
   at CommandLine.Parser.ParseArguments[T](IEnumerable`1)
   at delta_kusto.Program.Main(String[])

Should I create a new issue?

from delta-kusto.

vplauzon avatar vplauzon commented on May 24, 2024

Oh... that isn't good! That's an error we had during integration but was supposed to be gone.

I'm looking at it.

from delta-kusto.

jeremiah-carlin avatar jeremiah-carlin commented on May 24, 2024

Thanks! happy to test or provide debugging information if it helps.

from delta-kusto.

vplauzon avatar vplauzon commented on May 24, 2024

I think I know what it is...

from delta-kusto.

vplauzon avatar vplauzon commented on May 24, 2024

Let me know if that works.

https://github.com/microsoft/delta-kusto/releases/tag/0.11.1.116

from delta-kusto.

jeremiah-carlin avatar jeremiah-carlin commented on May 24, 2024

Thanks for the quick fix! The CLI no longer throws the above error but it appears there are still issues with parsing the create ingestion command.

In my results the json mappings appear to be truncated at the first single quote in the mapping definition. For example the mapping [{"column":"EventId","path":"$['EventId']","datatype":"","transform":""}] is output as [{"column":"EventId","path":"$[ when running through delta-kusto

As a separate note: In attempting to create a simple reproduction of this issue I realized the ingestion mapping script that comes back from .show database schema as csl script causes a syntax error if you attempt to invoke it directly in the dataexplorer editor (even without using delta-kusto to get the ingestion mapping script). The editor highlights the table in the following command:

.create-or-alter table _ingest_test ingestion json mapping _ingest_test_mapping ```
[{"column":"EventId","path":"$['EventId']","datatype":"","transform":""}]

from delta-kusto.

vplauzon avatar vplauzon commented on May 24, 2024

Thank you, I was easily able to repro the issue on my side. I've sent a bug report to the owner of that command. Hopefully it should be fixed soon.

from delta-kusto.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.