pjfanning commented on PR #2631:
URL: https://github.com/apache/drill/pull/2631#issuecomment-1236377485
@cgivre Github code scanning picked up on some of the pre-existing issues -
otherwise, the build seems ok
--
This is an automated message from the Apache Git Service.
To respond to the
pjfanning commented on PR #2631:
URL: https://github.com/apache/drill/pull/2631#issuecomment-1236340874
@cgivre I just removed the lgtm comments. A common pattern used in the code
that were highlighted as index out bounds risks - I think LGTM was right, so
I've made the same change in
pjfanning commented on PR #2631:
URL: https://github.com/apache/drill/pull/2631#issuecomment-1230765965
@cgivre thanks for sorting out the INFRA ticket. This PR is now really just
for removing a file that is no longer needed.
--
This is an automated message from the Apache Git Service.
pjfanning commented on PR #2631:
URL: https://github.com/apache/drill/pull/2631#issuecomment-1225917557
@cgivre I raised https://issues.apache.org/jira/browse/INFRA-23627
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and