Internalize ANTLR sources; exclude from API checks

partiql · Apr 30, 2024 · fc0d4f8 · fc0d4f8
1 parent 607c4c0
commit fc0d4f8
Show file tree

Hide file tree

Showing 15 changed files with 33 additions and 5,312 deletions.
diff --git a/...grades/v0.7-to-v0.8-upgrade/upgraded-examples/src/test/kotlin/examples/BreakingChanges.kt b/...grades/v0.7-to-v0.8-upgrade/upgraded-examples/src/test/kotlin/examples/BreakingChanges.kt
@@ -13,7 +13,7 @@ import org.partiql.lang.eval.ExprValueFactory
 import org.partiql.lang.planner.PlannerPassResult
 import org.partiql.lang.syntax.Parser
 import org.partiql.lang.syntax.PartiQLParserBuilder
-import org.partiql.parser.antlr.PartiQLTokens
+import org.partiql.parser.internal.antlr.PartiQLTokens
 import org.partiql.lang.types.CustomType
 import java.nio.charset.StandardCharsets
 import kotlin.test.Test

diff --git a/partiql-cli/src/main/kotlin/org/partiql/cli/shell/ShellHighlighter.kt b/partiql-cli/src/main/kotlin/org/partiql/cli/shell/ShellHighlighter.kt
@@ -23,8 +23,8 @@ import org.jline.reader.LineReader
 import org.jline.utils.AttributedString
 import org.jline.utils.AttributedStringBuilder
 import org.jline.utils.AttributedStyle
-import org.partiql.parser.antlr.PartiQLParser
-import org.partiql.parser.antlr.PartiQLTokens
+import org.partiql.parser.internal.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLTokens
 import java.nio.charset.StandardCharsets
 import java.util.regex.Pattern
 

diff --git a/partiql-lang/src/main/kotlin/org/partiql/lang/syntax/impl/PartiQLPigParser.kt b/partiql-lang/src/main/kotlin/org/partiql/lang/syntax/impl/PartiQLPigParser.kt
@@ -39,8 +39,8 @@ import org.partiql.lang.types.CustomType
 import org.partiql.lang.util.checkThreadInterrupted
 import org.partiql.lang.util.getAntlrDisplayString
 import org.partiql.lang.util.getIonValue
-import org.partiql.parser.antlr.PartiQLParser
-import org.partiql.parser.antlr.PartiQLTokens
+import org.partiql.parser.internal.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLTokens
 import java.io.InputStream
 import java.nio.channels.ClosedByInterruptException
 import java.nio.charset.StandardCharsets

diff --git a/partiql-lang/src/main/kotlin/org/partiql/lang/syntax/impl/PartiQLPigVisitor.kt b/partiql-lang/src/main/kotlin/org/partiql/lang/syntax/impl/PartiQLPigVisitor.kt
@@ -60,8 +60,8 @@ import org.partiql.lang.util.checkThreadInterrupted
 import org.partiql.lang.util.error
 import org.partiql.lang.util.getPrecisionFromTimeString
 import org.partiql.lang.util.unaryMinus
-import org.partiql.parser.antlr.PartiQLBaseVisitor
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLBaseVisitor
+import org.partiql.parser.internal.antlr.PartiQLParser
 import org.partiql.pig.runtime.SymbolPrimitive
 import org.partiql.value.datetime.DateTimeException
 import org.partiql.value.datetime.TimeZone

diff --git a/partiql-lang/src/main/kotlin/org/partiql/lang/util/AntlrUtilities.kt b/partiql-lang/src/main/kotlin/org/partiql/lang/util/AntlrUtilities.kt
@@ -32,7 +32,7 @@ import org.partiql.errors.ErrorCode
 import org.partiql.errors.Property
 import org.partiql.errors.PropertyValueMap
 import org.partiql.lang.syntax.ParserException
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLParser
 import java.math.BigInteger
 
 // workaround until ErrorAndErrorContexts no longer uses IonSystem

diff --git a/partiql-lang/src/test/kotlin/org/partiql/lang/errors/ParserErrorsTest.kt b/partiql-lang/src/test/kotlin/org/partiql/lang/errors/ParserErrorsTest.kt
@@ -20,7 +20,7 @@ import org.partiql.errors.ErrorCode
 import org.partiql.errors.Property
 import org.partiql.lang.syntax.PartiQLParserTestBase
 import org.partiql.lang.util.getAntlrDisplayString
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLParser
 
 class ParserErrorsTest : PartiQLParserTestBase() {
 

diff --git a/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserDateTimeTests.kt b/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserDateTimeTests.kt
@@ -12,7 +12,7 @@ import org.partiql.lang.domains.PartiqlAst
 import org.partiql.lang.domains.id
 import org.partiql.lang.util.getAntlrDisplayString
 import org.partiql.lang.util.to
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLParser
 
 class PartiQLParserDateTimeTests : PartiQLParserTestBase() {
 

diff --git a/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserExplainTest.kt b/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserExplainTest.kt
@@ -22,7 +22,7 @@ import org.partiql.lang.ION
 import org.partiql.lang.domains.PartiqlAst
 import org.partiql.lang.util.ArgumentsProviderBase
 import org.partiql.lang.util.getAntlrDisplayString
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLParser
 
 class PartiQLParserExplainTest : PartiQLParserTestBase() {
 

diff --git a/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserTest.kt b/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserTest.kt
@@ -29,7 +29,7 @@ import org.partiql.lang.ast.sourceLocation
 import org.partiql.lang.domains.PartiqlAst
 import org.partiql.lang.domains.id
 import org.partiql.lang.util.getAntlrDisplayString
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLParser
 import kotlin.concurrent.thread
 
 /**

diff --git a/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserWindowTests.kt b/partiql-lang/src/test/kotlin/org/partiql/lang/syntax/PartiQLParserWindowTests.kt
@@ -4,7 +4,7 @@ import org.junit.Test
 import org.partiql.errors.ErrorCode
 import org.partiql.errors.Property
 import org.partiql.lang.util.getAntlrDisplayString
-import org.partiql.parser.antlr.PartiQLParser
+import org.partiql.parser.internal.antlr.PartiQLParser
 
 class PartiQLParserWindowTests : PartiQLParserTestBase() {
 

diff --git a/...lang/src/test/kotlin/org/partiql/lang/syntax/impl/PartiQLPigParserThreadInterruptTests.kt b/...lang/src/test/kotlin/org/partiql/lang/syntax/impl/PartiQLPigParserThreadInterruptTests.kt
@@ -31,7 +31,7 @@ import org.partiql.lang.StepContext
 import org.partiql.lang.domains.PartiqlAst
 import org.partiql.lang.eval.CompileOptions
 import org.partiql.lang.eval.visitors.VisitorTransformBase
-import org.partiql.parser.antlr.PartiQLTokens
+import org.partiql.parser.internal.antlr.PartiQLTokens
 import java.io.InputStream
 import java.util.concurrent.atomic.AtomicBoolean
 import kotlin.concurrent.thread

diff --git a/partiql-parser/README.adoc b/partiql-parser/README.adoc
@@ -3,6 +3,9 @@
 The PartiQL Parser can be used to parse PartiQL queries into an AST (`org.partiql.ast`).
 This interface expands the legacy `org.partiql.syntax.Parser` interface allowing for a richer return type as well as the latest AST.
 
+ANTLR-generated code is under the `internal` package namespace. It is not intended to be used outside of code within
+this library.
+
 == Interfaces
 
 [source,kotlin]