[incubator-druid] branch master updated: SQLMetadataSegmentManager: Comments, formatting adjustments (#7452)

This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 3854cfd SQLMetadataSegmentManager:

[GitHub] [incubator-druid] fjy merged pull request #7456: Allow the console config to pass in a custom header name and value

fjy merged pull request #7456: Allow the console config to pass in a custom header name and value URL: https://github.com/apache/incubator-druid/pull/7456 This is an automated message from the Apache Git Service. To respond

[incubator-druid] branch master updated: customHeader (#7456)

This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 07eb2ef customHeader (#7456) 07eb2ef is

[GitHub] [incubator-druid] fjy merged pull request #7452: SQLMetadataSegmentManager: Comments, formatting adjustments

fjy merged pull request #7452: SQLMetadataSegmentManager: Comments, formatting adjustments URL: https://github.com/apache/incubator-druid/pull/7452 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] fjy merged pull request #7455: SQL: Fix docs for STRING_FORMAT.

fjy merged pull request #7455: SQL: Fix docs for STRING_FORMAT. URL: https://github.com/apache/incubator-druid/pull/7455 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-druid] love1693294577 commented on issue #7411: [materialized view] Too many Hadoop jobs doing the same thing

love1693294577 commented on issue #7411: [materialized view] Too many Hadoop jobs doing the same thing URL: https://github.com/apache/incubator-druid/issues/7411#issuecomment-482428382 hello，materialized-view get all derivatives for datasource in query is null ， Have you ever met?

[GitHub] [incubator-druid] love1693294577 commented on issue #7403: how to use MaterializedView ，I didn't find the relevant submit task command.

love1693294577 commented on issue #7403: how to use MaterializedView ，I didn't find the relevant submit task command. URL: https://github.com/apache/incubator-druid/issues/7403#issuecomment-482427744 > Hi, > > You have to run something like this: > > ``` > curl -X 'POST'

[GitHub] [incubator-druid] vogievetsky opened a new pull request #7456: Allow the console config to pass in a custom header name and value

vogievetsky opened a new pull request #7456: Allow the console config to pass in a custom header name and value URL: https://github.com/apache/incubator-druid/pull/7456 This can be used to set an auth header if Druid is running in secured mode

[GitHub] [incubator-druid] gianm opened a new pull request #7455: SQL: Fix docs for STRING_FORMAT.

gianm opened a new pull request #7455: SQL: Fix docs for STRING_FORMAT. URL: https://github.com/apache/incubator-druid/pull/7455 I got the name of the function wrong in #7327. This is an automated message from the Apache Git

[GitHub] [incubator-druid] jon-wei commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

jon-wei commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274728196 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/schema/SystemSchema.java ##

[GitHub] [incubator-druid] justinborromeo opened a new pull request #7454: Support querying realtime segments using time-ordered scan queries

justinborromeo opened a new pull request #7454: Support querying realtime segments using time-ordered scan queries URL: https://github.com/apache/incubator-druid/pull/7454 #7373 introduced a bug where unpublished realtime segments were unqueryable when ordering on time using scan queries.

[GitHub] [incubator-druid] clintropolis closed issue #7156: web console 'tasks' view group tasks by 'status' broken

clintropolis closed issue #7156: web console 'tasks' view group tasks by 'status' broken URL: https://github.com/apache/incubator-druid/issues/7156 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [incubator-druid] clintropolis merged pull request #7440: Fixed grouping and sorting by status in task table

clintropolis merged pull request #7440: Fixed grouping and sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7440 This is an automated message from the Apache Git Service. To respond to the

[incubator-druid] branch master updated: Fixed grouping and sorting by status in task table (#7440)

This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 7382dac Fixed grouping and sorting by

[GitHub] [incubator-druid] vogievetsky commented on issue #7449: Wrap query with limit within the web console

vogievetsky commented on issue #7449: Wrap query with limit within the web console URL: https://github.com/apache/incubator-druid/pull/7449#issuecomment-482354638 @fjy noted: made the button more explicit (see updated screenshot in description)

[GitHub] [incubator-druid] vogievetsky commented on issue #7440: Fixed grouping and sorting by status in task table

vogievetsky commented on issue #7440: Fixed grouping and sorting by status in task table URL: https://github.com/apache/incubator-druid/pull/7440#issuecomment-482349617 Looks great, thank you  This is an automated message

[GitHub] [incubator-druid] vogievetsky commented on issue #7451: Fix datasource availability sorting

vogievetsky commented on issue #7451: Fix datasource availability sorting URL: https://github.com/apache/incubator-druid/pull/7451#issuecomment-482349313 This looks great thank you  This is an automated message from the

[GitHub] [incubator-druid] gianm commented on a change in pull request #7281: bump calcite version to 1.18.0 in druid-sql

gianm commented on a change in pull request #7281: bump calcite version to 1.18.0 in druid-sql URL: https://github.com/apache/incubator-druid/pull/7281#discussion_r274678384 ## File path: sql/src/test/java/org/apache/druid/sql/calcite/CalciteQueryTest.java ## @@ -1737,23

[incubator-druid] branch master updated: adding host to the error display (#7453)

This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new c5c2bf7 adding host to the error display

[GitHub] [incubator-druid] fjy merged pull request #7453: Adding 'host' to the error display in web console

fjy merged pull request #7453: Adding 'host' to the error display in web console URL: https://github.com/apache/incubator-druid/pull/7453 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-druid] surekhasaharan commented on issue #7350: Remove unnecessary collection

surekhasaharan commented on issue #7350: Remove unnecessary collection URL: https://github.com/apache/incubator-druid/pull/7350#issuecomment-482329733 @leventov @drcrallen do you have any more comments on this ? This is an

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274644465 ## File path: core/src/main/java/org/apache/druid/timeline/SegmentWithOvershadowedStatus.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274653345 ## File path: server/src/main/java/org/apache/druid/server/http/MetadataResource.java

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table

surekhasaharan commented on a change in pull request #7425: Add is_overshadowed column to sys.segments table URL: https://github.com/apache/incubator-druid/pull/7425#discussion_r274645284 ## File path: docs/content/querying/sql.md ## @@ -609,6 +609,7 @@ Note that a

[GitHub] [incubator-druid] surekhasaharan commented on a change in pull request #7325: Fix result-level cache for queries

surekhasaharan commented on a change in pull request #7325: Fix result-level cache for queries URL: https://github.com/apache/incubator-druid/pull/7325#discussion_r274641008 ## File path: processing/src/main/java/org/apache/druid/query/groupby/having/GreaterThanHavingSpec.java

[GitHub] [incubator-druid] gianm merged pull request #7313: chore: pin upstream java container to jdk 8

gianm merged pull request #7313: chore: pin upstream java container to jdk 8 URL: https://github.com/apache/incubator-druid/pull/7313 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-druid] branch master updated: chore: pin upstream java container to jdk 8 (#7313)

This is an automated email from the ASF dual-hosted git repository. gian pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 08d0d6d chore: pin upstream java

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7388: Support LPAD and RPAD sql function

clintropolis commented on a change in pull request #7388: Support LPAD and RPAD sql function URL: https://github.com/apache/incubator-druid/pull/7388#discussion_r274631667 ## File path: docs/content/misc/math-expr.md ## @@ -85,6 +85,8 @@ The following built-in functions

[GitHub] [incubator-druid] gianm commented on issue #6581: Moments Sketch custom aggregator

gianm commented on issue #6581: Moments Sketch custom aggregator URL: https://github.com/apache/incubator-druid/pull/6581#issuecomment-482289512 @edgan8 This'll be available as a contrib extension starting in 0.15.0. No need to do anything on your end!

[GitHub] [incubator-druid] capistrant commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication

capistrant commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication URL: https://github.com/apache/incubator-druid/pull/7145#discussion_r274603797 ## File path:

[GitHub] [incubator-druid] capistrant commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication

capistrant commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication URL: https://github.com/apache/incubator-druid/pull/7145#discussion_r274603707 ## File path:

[GitHub] [incubator-druid] mohammadjkhan commented on issue #6972: Support LDAP authentication/authorization

mohammadjkhan commented on issue #6972: Support LDAP authentication/authorization URL: https://github.com/apache/incubator-druid/pull/6972#issuecomment-482270756 @jon-wei * LDAP is supposed to be source of truth for users/groups and not roles/permissions. The standard way to implement

[GitHub] [incubator-druid] capistrant commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication

capistrant commented on a change in pull request #7145: Enhance the Http Firehose to work with URIs requiring basic authentication URL: https://github.com/apache/incubator-druid/pull/7145#discussion_r274603656 ## File path: docs/content/ingestion/firehose.md ## @@ -74,6

[GitHub] [incubator-druid] vogievetsky opened a new pull request #7453: Adding 'host' to the error display in web console

vogievetsky opened a new pull request #7453: Adding 'host' to the error display in web console URL: https://github.com/apache/incubator-druid/pull/7453 Because host is important also.

[GitHub] [incubator-druid] fjy commented on issue #7449: Wrap query with limit within the web console

fjy commented on issue #7449: Wrap query with limit within the web console URL: https://github.com/apache/incubator-druid/pull/7449#issuecomment-482261214 @vogievetsky this needs to be more obvious, i.e. right beside the run button

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274590624 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ##

[GitHub] [incubator-druid] leventov commented on issue #7447: Coordinator: Allow dropping all segments.

leventov commented on issue #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#issuecomment-482249849 The design with `SQLMetadataSegmentManager` enforcing leadership changes on its callers doesn't feel right to me. It forces all

[GitHub] [incubator-druid] leventov edited a comment on issue #7447: Coordinator: Allow dropping all segments.

leventov edited a comment on issue #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#issuecomment-482249849 The design with `SQLMetadataSegmentManager` enforcing leadership changes on its callers doesn't feel right to me. It forces

[GitHub] [incubator-druid] leerho commented on issue #7444: Are there documentation on the limitation of intersection using theta sketch?

leerho commented on issue #7444: Are there documentation on the limitation of intersection using theta sketch? URL: https://github.com/apache/incubator-druid/issues/7444#issuecomment-482247719 Refer to documentation [here](https://datasketches.github.io/docs/Theta/ThetaSketchSetOps.html)

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274575999 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ## @@

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274575456 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ## @@

[GitHub] [incubator-druid] gianm opened a new pull request #7452: SQLMetadataSegmentManager: Comments, formatting adjustments

gianm opened a new pull request #7452: SQLMetadataSegmentManager: Comments, formatting adjustments URL: https://github.com/apache/incubator-druid/pull/7452 Follow up to #7447. This is an automated message from the Apache Git

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274575416 ## File path: server/src/test/java/org/apache/druid/metadata/SQLMetadataSegmentManagerTest.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274575149 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ##

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274567128 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ## @@

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274566247 ## File path: server/src/test/java/org/apache/druid/metadata/SQLMetadataSegmentManagerTest.java ##

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274565498 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ## @@

[GitHub] [incubator-druid] gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

gianm commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274563782 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ## @@

[GitHub] [incubator-druid] shuqi7 opened a new pull request #7451: Fix datasource availability sorting

shuqi7 opened a new pull request #7451: Fix datasource availability sorting URL: https://github.com/apache/incubator-druid/pull/7451 Now the availability column in datasource table could be sorted by 1. The availability percentage 2. The total number of segments if the percentage ties

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r274546791 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r274542553 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments

leventov commented on a change in pull request #7306: Reconcile terminology and method naming to 'used/unused segments'; Rename MetadataSegmentManager to MetadataSegments URL: https://github.com/apache/incubator-druid/pull/7306#discussion_r274540701 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274535009 ## File path:

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274528147 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274538904 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ##

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274530317 ## File path: server/src/test/java/org/apache/druid/metadata/SQLMetadataSegmentManagerTest.java

[GitHub] [incubator-druid] leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

leventov commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274525903 ## File path: server/src/main/java/org/apache/druid/metadata/SQLMetadataSegmentManager.java ##

[GitHub] [incubator-druid] leventov commented on issue #7447: Coordinator: Allow dropping all segments.

leventov commented on issue #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#issuecomment-482201538 I think this PR should had a `Design Review` tag. Such PRs shouldn't be merged 10 hours after opening.

[GitHub] [incubator-druid] leventov commented on issue #7433: Explain a difference between interval and trueInterval in TimelineObjectHolder

leventov commented on issue #7433: Explain a difference between interval and trueInterval in TimelineObjectHolder URL: https://github.com/apache/incubator-druid/issues/7433#issuecomment-482196655 There should be Javadoc links to the explanation from the fields of TimelineObjectHolder.

[incubator-druid] branch master updated: Coordinator: Allow dropping all segments. (#7447)

This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new a517f8c Coordinator: Allow dropping all

[GitHub] [incubator-druid] nrwiersma opened a new issue #7450: SQL: Support "subtotalsSpec"

nrwiersma opened a new issue #7450: SQL: Support "subtotalsSpec" URL: https://github.com/apache/incubator-druid/issues/7450 ### Description subtotalsSpec which is closest to `GROUPING SET` makes it far more efficient to create multiple groups that without it, would need to be done

[GitHub] [incubator-druid] love1693294577 closed issue #7403: how to use MaterializedView ，I didn't find the relevant submit task command.

love1693294577 closed issue #7403: how to use MaterializedView ，I didn't find the relevant submit task command. URL: https://github.com/apache/incubator-druid/issues/7403 This is an automated message from the Apache Git

[GitHub] [incubator-druid] clintropolis merged pull request #7421: Add time taken by last query in SQL view

clintropolis merged pull request #7421: Add time taken by last query in SQL view URL: https://github.com/apache/incubator-druid/pull/7421 This is an automated message from the Apache Git Service. To respond to the message,

[incubator-druid] branch master updated: Add time taken by last query in SQL view (#7421)

This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 4ea37e2 Add time taken by last query

[GitHub] [incubator-druid] clintropolis merged pull request #7435: Table pagination style change

clintropolis merged pull request #7435: Table pagination style change URL: https://github.com/apache/incubator-druid/pull/7435 This is an automated message from the Apache Git Service. To respond to the message, please log

[incubator-druid] branch master updated: Table pagination style change (#7435)

This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new bbb6201 Table pagination style change

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7447: Coordinator: Allow dropping all segments.

clintropolis commented on a change in pull request #7447: Coordinator: Allow dropping all segments. URL: https://github.com/apache/incubator-druid/pull/7447#discussion_r274274442 ## File path: