[PR] Wait for SQL table visibility in embedded tests. (druid)

["gianm (via GitHub)" <gi...@apache.org>]

gianm opened a new pull request, #19095:
URL: https://github.com/apache/druid/pull/19095

   Several embedded tests wait for segment/schemaCache/refresh/count as a proxy for when a datasource is queryable via SQL. However, this metric is emitted before the SQL table is set up. This creates a window where SQL queries can fail with "Object 'datasource' not found".
   
   This patch waits for segment/schemaCache/rowSignature/column/count, which is emitted after a SQL table is set up and visible. This is done either as a replacement for segment/schemaCache/refresh/count (if we only care about table visibility) or in addition to it (if we also want to wait for a specific number of segments to be detected).


-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org

Re: [PR] Wait for SQL table visibility in embedded tests. (druid)

["gianm (via GitHub)" <gi...@apache.org>]

gianm merged PR #19095:
URL: https://github.com/apache/druid/pull/19095


-- 
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: commits-unsubscribe@druid.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org