substrait-io / substrait-java

Apache License 2.0
75 stars 72 forks source link

build: use antlr4-runtime instead of antlr #182

Closed guiyanakuang closed 1 year ago

guiyanakuang commented 1 year ago

To avoid introducing unnecessary indirect dependencies. Resolve issue #162

vbarua commented 1 year ago

I'm seeing build failures on this, but I don't think they're related to your changes. I'm going to investigate them today.

guiyanakuang commented 1 year ago

@vbarua . Thanks for the quick review and merge of PR.