Skip to content

Conversation

@lowka
Copy link
Contributor

@lowka lowka commented Jan 30, 2023

No description provided.

@lowka lowka force-pushed the IGNITE-15979 branch 8 times, most recently from c45fac3 to c94d09b Compare January 31, 2023 09:44
@lowka lowka force-pushed the IGNITE-15979 branch 2 times, most recently from 5cfacc1 to 64b51c7 Compare February 3, 2023 11:36
@AMashenkov
Copy link
Member

Looks good.
Les't fix styles and extract static Command classes from SqlScriptRunner to the top-level classes.

@lowka
Copy link
Contributor Author

lowka commented Feb 6, 2023

Looks good.
Les't fix styles and extract static Command classes from SqlScriptRunner to the top-level classes.

Done + added javadocs and some test cases to _runner/self.test

@korlov42 korlov42 merged commit 343026e into apache:main Feb 6, 2023
@korlov42 korlov42 deleted the IGNITE-15979 branch February 6, 2023 11:13
lowka added a commit to gridgain/apache-ignite-3 that referenced this pull request Mar 18, 2023
lowka added a commit to gridgain/apache-ignite-3 that referenced this pull request Apr 19, 2023
isapego pushed a commit to isapego/ignite-3 that referenced this pull request Dec 26, 2024
* Add tests for Avro, Protobuf, JSON Schema, JSON serializers
* Assign names and versions to generated schemas
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants