You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by "vvysotskyi (via GitHub)" <gi...@apache.org> on 2023/02/09 09:31:15 UTC
[GitHub] [drill] vvysotskyi opened a new pull request, #2752: DRILL-8397: Drill prints warnings to console when starting it
vvysotskyi opened a new pull request, #2752:
URL: https://github.com/apache/drill/pull/2752
# [DRILL-8397](https://issues.apache.org/jira/browse/DRILL-8397): Drill prints warnings to console when starting it
## Description
Replaced the `level` tag with the attribute, so no extra output is printed to the console.
## Documentation
NA
## Testing
Checked manually
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] jnturton merged pull request #2752: DRILL-8397: Drill prints warnings to console when starting it
Posted by "jnturton (via GitHub)" <gi...@apache.org>.
jnturton merged PR #2752:
URL: https://github.com/apache/drill/pull/2752
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] cgivre commented on pull request #2752: DRILL-8397: Drill prints warnings to console when starting it
Posted by "cgivre (via GitHub)" <gi...@apache.org>.
cgivre commented on PR #2752:
URL: https://github.com/apache/drill/pull/2752#issuecomment-1424151052
>
Given the flakyness of that test, I'm fine with marking that as a "run manually" test. I think you are correct that the indexing doesn't happen immediately.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] jnturton commented on pull request #2752: DRILL-8397: Drill prints warnings to console when starting it
Posted by "jnturton (via GitHub)" <gi...@apache.org>.
jnturton commented on PR #2752:
URL: https://github.com/apache/drill/pull/2752#issuecomment-1424093562
Regarding the sporadic failures of the unit test testBasicCTASWithScalarDataTypes that have been showing in recent CI runs, and now also here, it looks to me like [the Thread.sleep(15_000) that it uses](https://github.com/jnturton/drill/blob/83bf04cd815e6c6d22d9ef04550cd8113bd9f905/contrib/storage-splunk/src/test/java/org/apache/drill/exec/store/splunk/SplunkWriterTest.java#L113) to wait for an INSERT to go through is not always enough to ensure that Splunk will return the record needed for the subsequent schema validation to pass.
Locally I've been able to reproduce both successes and failures of this test with a range of sleep durations from 6_000 to 60_000 so I'm not sure if there is a magic number here, or what it might be. Stepping back further, I don't know if Splunk ever takes a transactional approach to loading data - maybe it makes no guarantees beyond "Thanks, I'll index this later"?
CC @cgivre.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [drill] jnturton commented on pull request #2752: DRILL-8397: Drill prints warnings to console when starting it
Posted by "jnturton (via GitHub)" <gi...@apache.org>.
jnturton commented on PR #2752:
URL: https://github.com/apache/drill/pull/2752#issuecomment-1424101087
In any case, that failure is certainly unrelated to this PR which in my opinion may be merged.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscribe@drill.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org