[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-08-17 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1296921605 ## plus/src/test/java/org/apache/calcite/slt/SqlLogicTests.java: ## @@ -0,0 +1,337 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-08-17 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1296913798 ## plus/src/test/java/org/apache/calcite/slt/SqlLogicTests.java: ## @@ -0,0 +1,337 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-08-17 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1296908112 ## plus/src/test/java/org/apache/calcite/slt/SqlLogicTests.java: ## @@ -0,0 +1,337 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-08-17 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1296907558 ## plus/src/test/java/org/apache/calcite/slt/SqlLogicTests.java: ## @@ -0,0 +1,337 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-08-17 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1296836281 ## plus/src/test/java/org/apache/calcite/slt/SqlLogicTests.java: ## @@ -0,0 +1,337 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-07-17 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1265184045 ## plus/src/test/java/org/apache/calcite/slt/README.md: ## @@ -0,0 +1,130 @@ + + +# Testing Calcite using SQL Logic Tests + +[SQL Logic

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-05-25 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1205976037 ## plus/src/test/java/org/apache/calcite/slt/executors/CalciteExecutor.java: ## @@ -0,0 +1,170 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-05-25 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1205973836 ## plus/src/test/java/org/apache/calcite/slt/TestCalcite.java: ## @@ -0,0 +1,86 @@ +/* + * Licensed to the Apache Software Foundation (ASF) under one or more + *

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-05-23 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1201939308 ## plus/build.gradle.kts: ## @@ -25,9 +25,11 @@ dependencies { implementation("com.teradata.tpcds:tpcds") implementation("io.prestosql.tpch:tpch")

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-05-05 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1186452908 ## slt/README.md: ## @@ -0,0 +1,72 @@ + + +# SQL logic tests Review Comment: We are not going to distribute any `slt` binaries as part of Calcite so we don't need

[GitHub] [calcite] zabetak commented on a diff in pull request #3145: [CALCITE-5615] Program to run SQL Logic Tests for Calcite

2023-04-14 Thread via GitHub
zabetak commented on code in PR #3145: URL: https://github.com/apache/calcite/pull/3145#discussion_r1166963052 ## slt/src/main/java/org/apache/calcite/slt/Main.java: ## @@ -0,0 +1,208 @@ +/* + * Copyright 2022 VMware, Inc. + * SPDX-License-Identifier: MIT + *