[GitHub] [drill] jnturton commented on a diff in pull request #2609: DRILL-8267: Remove commons-configuration dependency management

2022-07-20 Thread GitBox
jnturton commented on code in PR #2609: URL: https://github.com/apache/drill/pull/2609#discussion_r925256805 ## pom.xml: ## @@ -1984,17 +1983,6 @@ xercesImpl ${xerces.version} - Review Comment: Note that what we're doing here is removing the _

[GitHub] [drill] jnturton commented on a diff in pull request #2609: DRILL-8267: Remove commons-configuration dependency management

2022-07-21 Thread GitBox
jnturton commented on code in PR #2609: URL: https://github.com/apache/drill/pull/2609#discussion_r926359954 ## pom.xml: ## @@ -1984,17 +1983,6 @@ xercesImpl ${xerces.version} - Review Comment: Okay @pjfanning, based on @vdiravka's comments I