[GitHub] [druid] jihoonson commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
jihoonson commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708886564 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through a

[GitHub] [druid] zachjsh commented on a change in pull request #11692: refactor sql authorization to get resource type from schema, resource type to be string

2021-09-14 Thread GitBox
zachjsh commented on a change in pull request #11692: URL: https://github.com/apache/druid/pull/11692#discussion_r708882212 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/SqlResourceCollectorShuttle.java ## @@ -69,10 +68,9 @@ public SqlNode visit(SqlIdent

[GitHub] [druid] zachjsh commented on a change in pull request #11692: refactor sql authorization to get resource type from schema, resource type to be string

2021-09-14 Thread GitBox
zachjsh commented on a change in pull request #11692: URL: https://github.com/apache/druid/pull/11692#discussion_r708873102 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerContext.java ## @@ -179,6 +185,12 @@ public DateTimeZone getTimeZone() r

[GitHub] [druid] FrankChen021 commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
FrankChen021 commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708858776 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through

[GitHub] [druid] jihoonson commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
jihoonson commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708820878 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through a

[GitHub] [druid] jihoonson commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
jihoonson commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708820878 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through a

[GitHub] [druid] FrankChen021 commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
FrankChen021 commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708772162 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through

[GitHub] [druid] jihoonson commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
jihoonson commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708755276 ## File path: docs/querying/sql.md ## @@ -859,8 +859,15 @@ the SQL planner. ### HTTP POST -You can make Druid SQL queries using HTTP via POST to th

[GitHub] [druid] clintropolis opened a new pull request #11709: dependency check with inhert instead of aggregate

2021-09-14 Thread GitBox
clintropolis opened a new pull request #11709: URL: https://github.com/apache/druid/pull/11709 I'm having some trouble building release artifacts due to the dependency check failing because it tries to find CVE in `druid-integration-tests` (which is unnecessary). This is just undoing the c

[druid] branch master updated: Fix truncation detectability for SQL array, object formats. (#11685)

2021-09-14 Thread gian
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/druid.git The following commit(s) were added to refs/heads/master by this push: new 7220d04 Fix truncation detectability for SQL array

[GitHub] [druid] gianm merged pull request #11685: Fix truncation detectability for SQL array, object formats.

2021-09-14 Thread GitBox
gianm merged pull request #11685: URL: https://github.com/apache/druid/pull/11685 -- 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-unsubscr

[GitHub] [druid] techdocsmith commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
techdocsmith commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708648327 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through

[GitHub] [druid] vtlim commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
vtlim commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708631246 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through a JSON

[GitHub] [druid] techdocsmith commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
techdocsmith commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708635568 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through

[GitHub] [druid] jihoonson commented on a change in pull request #11708: docs: delete / cancel query

2021-09-14 Thread GitBox
jihoonson commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708622787 ## File path: docs/querying/sql.md ## @@ -956,6 +955,30 @@ trailer they all include: one blank line at the end of the result set. If you de through a

[GitHub] [druid-website] vogievetsky merged pull request #139: Autobuild

2021-09-14 Thread GitBox
vogievetsky merged pull request #139: URL: https://github.com/apache/druid-website/pull/139 -- 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: commit

[druid-website] branch asf-site updated: Autobuild (#139)

2021-09-14 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a commit to branch asf-site in repository https://gitbox.apache.org/repos/asf/druid-website.git The following commit(s) were added to refs/heads/asf-site by this push: new 1656163 Autobuild (#139) 165616

[GitHub] [druid-website-src] vogievetsky merged pull request #253: small changes to messaging

2021-09-14 Thread GitBox
vogievetsky merged pull request #253: URL: https://github.com/apache/druid-website-src/pull/253 -- 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: co

[druid-website-src] branch master updated (b32d160 -> 79f0d13)

2021-09-14 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a change to branch master in repository https://gitbox.apache.org/repos/asf/druid-website-src.git. from b32d160 Merge pull request #250 from petermarshallio/202109-fArticles add 625fb6f small change

[druid-website-src] 01/01: Merge pull request #253 from apache/nn

2021-09-14 Thread vogievetsky
This is an automated email from the ASF dual-hosted git repository. vogievetsky pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/druid-website-src.git commit 79f0d13d9e1c32e3fd80bbe410d0efc09ad01c88 Merge: b32d160 625fb6f Author: Vadim Ogievetsky AuthorDate: Tue

[druid-website-src] branch nn created (now 625fb6f)

2021-09-14 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a change to branch nn in repository https://gitbox.apache.org/repos/asf/druid-website-src.git. at 625fb6f small changes to messaging This branch includes the following new commits: new 625fb6f small cha

[druid-website-src] 01/01: small changes to messaging

2021-09-14 Thread fjy
This is an automated email from the ASF dual-hosted git repository. fjy pushed a commit to branch nn in repository https://gitbox.apache.org/repos/asf/druid-website-src.git commit 625fb6fdfd2670337bef147fde7bb503d78fa12d Author: fjy AuthorDate: Tue Sep 14 13:28:40 2021 -0700 small changes t

[GitHub] [druid] clintropolis commented on a change in pull request #11692: refactor sql authorization to get resource type from schema, resource type to be string

2021-09-14 Thread GitBox
clintropolis commented on a change in pull request #11692: URL: https://github.com/apache/druid/pull/11692#discussion_r708605582 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerContext.java ## @@ -179,6 +185,12 @@ public DateTimeZone getTimeZone()

[GitHub] [druid] techdocsmith commented on a change in pull request #11708: draft delete query

2021-09-14 Thread GitBox
techdocsmith commented on a change in pull request #11708: URL: https://github.com/apache/druid/pull/11708#discussion_r708572781 ## File path: docs/querying/sql.md ## @@ -956,6 +955,24 @@ trailer they all include: one blank line at the end of the result set. If you de through

[GitHub] [druid] techdocsmith opened a new pull request #11708: draft delete query

2021-09-14 Thread GitBox
techdocsmith opened a new pull request #11708: URL: https://github.com/apache/druid/pull/11708 Adds documentation for the DELETE method to cancel queries. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL ab

[GitHub] [druid] clintropolis commented on issue #11690: authorization by string based resource type instead of enum so that resource types are extensible

2021-09-14 Thread GitBox
clintropolis commented on issue #11690: URL: https://github.com/apache/druid/issues/11690#issuecomment-919426759 >What disruption are you concerned about exactly; as far as I understand the Resource class is not extendable as of now, so are there are extensions that would be disrupted by t

[GitHub] [druid] zachjsh commented on a change in pull request #11692: refactor sql authorization to get resource type from schema, resource type to be string

2021-09-14 Thread GitBox
zachjsh commented on a change in pull request #11692: URL: https://github.com/apache/druid/pull/11692#discussion_r708540274 ## File path: sql/src/main/java/org/apache/druid/sql/calcite/planner/PlannerContext.java ## @@ -179,6 +185,12 @@ public DateTimeZone getTimeZone() r

[GitHub] [druid] zachjsh commented on a change in pull request #11692: refactor sql authorization to get resource type from schema, resource type to be string

2021-09-14 Thread GitBox
zachjsh commented on a change in pull request #11692: URL: https://github.com/apache/druid/pull/11692#discussion_r708532764 ## File path: extensions-core/druid-basic-security/src/test/java/org/apache/druid/security/BasicAuthUtilsTest.java ## @@ -92,7 +93,7 @@ public void test

[GitHub] [druid] jon-wei commented on a change in pull request #11688: Task reports for parallel task: single phase and sequential mode

2021-09-14 Thread GitBox
jon-wei commented on a change in pull request #11688: URL: https://github.com/apache/druid/pull/11688#discussion_r708519553 ## File path: indexing-service/src/main/java/org/apache/druid/indexing/common/task/batch/parallel/ParallelIndexSupervisorTask.java ## @@ -169,6 +173,11 @

[GitHub] [druid] zachjsh edited a comment on issue #11690: authorization by string based resource type instead of enum so that resource types are extensible

2021-09-14 Thread GitBox
zachjsh edited a comment on issue #11690: URL: https://github.com/apache/druid/issues/11690#issuecomment-919392504 @clintropolis this approach sounds great. One question, you noted that This approach does provide some minor disruption for extension writers, as Resource.getType

[GitHub] [druid] zachjsh commented on issue #11690: authorization by string based resource type instead of enum so that resource types are extensible

2021-09-14 Thread GitBox
zachjsh commented on issue #11690: URL: https://github.com/apache/druid/issues/11690#issuecomment-919392504 @clintropolis this approach sounds great. One question, you noted that ``` This approach does provide some minor disruption for extension writers, as Resource.getType will

[GitHub] [druid] viongpanzi commented on issue #11700: VersionedIntervalTimeline.isOvershadowed is very slow when processing many segments

2021-09-14 Thread GitBox
viongpanzi commented on issue #11700: URL: https://github.com/apache/druid/issues/11700#issuecomment-919320856 @jihoonson Thank you for your explanation and advice! Our data volume is too large, we need to keep it to be daily-partitioned in order to save more disks. Now the Coordinator has

[druid] branch 0.22.0 updated: remove helm chart stuffs pending ip-clearance

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.22.0 by this push: new 9dfc259 remove helm chart stuffs pending ip-clea

[druid] branch 0.22.0 updated: Docs - ingestion task log config and process (#11678) (#11707)

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.22.0 by this push: new 5d02179 Docs - ingestion task log config and pro

[druid] branch 0.22.0 updated: add clarification for maxSubqueryRows (#11687) (#11706)

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.22.0 by this push: new 045fa98 add clarification for maxSubqueryRows (#

[GitHub] [druid] clintropolis merged pull request #11707: [Backport] Docs - ingestion task log config and process

2021-09-14 Thread GitBox
clintropolis merged pull request #11707: URL: https://github.com/apache/druid/pull/11707 -- 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-u

[druid] branch 0.22.0 updated: Docs - granularities link back to segmentGranularity (#11672) (#11705)

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.22.0 by this push: new b3b4a54 Docs - granularities link back to segmen

[GitHub] [druid] clintropolis merged pull request #11706: [Backport] add clarification for maxSubqueryRows

2021-09-14 Thread GitBox
clintropolis merged pull request #11706: URL: https://github.com/apache/druid/pull/11706 -- 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-u

[druid] branch 0.22.0 updated (989dcdc -> 23b58e5)

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a change to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git. from 989dcdc fix segment timeline in FireFox (#11701) (#11703) add 23b58e5 Web console: Account for segment granular

[GitHub] [druid] clintropolis merged pull request #11705: [Backport] Docs - granularities link back to segmentGranularity

2021-09-14 Thread GitBox
clintropolis merged pull request #11705: URL: https://github.com/apache/druid/pull/11705 -- 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-u

[druid] branch 0.22.0 updated: fix segment timeline in FireFox (#11701) (#11703)

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.22.0 by this push: new 989dcdc fix segment timeline in FireFox (#11701)

[GitHub] [druid] clintropolis merged pull request #11704: [Backport] Web console: Account for segment granularity all

2021-09-14 Thread GitBox
clintropolis merged pull request #11704: URL: https://github.com/apache/druid/pull/11704 -- 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-u

[GitHub] [druid] clintropolis merged pull request #11703: [Backport] Web console: fix segment timeline in Firefox

2021-09-14 Thread GitBox
clintropolis merged pull request #11703: URL: https://github.com/apache/druid/pull/11703 -- 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-u

[druid] branch 0.22.0 updated: suppress false positive cve (#11699) (#11702)

2021-09-14 Thread cwylie
This is an automated email from the ASF dual-hosted git repository. cwylie pushed a commit to branch 0.22.0 in repository https://gitbox.apache.org/repos/asf/druid.git The following commit(s) were added to refs/heads/0.22.0 by this push: new 5c59d77 suppress false positive cve (#11699) (#1

[GitHub] [druid] clintropolis merged pull request #11702: [Backport] suppress false positive cve

2021-09-14 Thread GitBox
clintropolis merged pull request #11702: URL: https://github.com/apache/druid/pull/11702 -- 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-u

[GitHub] [druid] clintropolis commented on a change in pull request #11650: add MV_FILTER_ONLY, MV_FILTER_NONE, ListFilteredVirtualColumn

2021-09-14 Thread GitBox
clintropolis commented on a change in pull request #11650: URL: https://github.com/apache/druid/pull/11650#discussion_r708034339 ## File path: processing/src/main/java/org/apache/druid/segment/virtual/ListFilteredVirtualColumn.java ## @@ -0,0 +1,193 @@ +/* + * Licensed to the