[GitHub] [incubator-druid] jihoonson commented on issue #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson commented on issue #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#issuecomment-507005800 > To clarify, are the contents of the generated LICENSE.BINARY currently the same, just not the same format? Yes, the contents should

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298817390 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed to

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298817393 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed to

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298817403 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed to

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298817393 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed to

[GitHub] [incubator-druid] jihoonson commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298817388 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed to

[GitHub] [incubator-druid] khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298806995 ## File path: .travis.yml ## @@ -51,7 +51,7 @@ matrix: - env: -

[GitHub] [incubator-druid] clintropolis commented on issue #7998: Binary license management system

2019-06-29 Thread GitBox
clintropolis commented on issue #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#issuecomment-506999704 >The generated LICENSE.BINARY file is a little bit different from the current one, mostly because of the difficulty of auto generation.

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298815422 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298815407 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298815390 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7998: Binary license management system

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#discussion_r298815393 ## File path: docs/_bin/generate-license.py ## @@ -0,0 +1,696 @@ +#!/usr/bin/env python3 + +# Licensed

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7547: Add support minor compaction with segment locking

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7547: Add support minor compaction with segment locking URL: https://github.com/apache/incubator-druid/pull/7547#discussion_r298815228 ## File path:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7547: Add support minor compaction with segment locking

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7547: Add support minor compaction with segment locking URL: https://github.com/apache/incubator-druid/pull/7547#discussion_r298815113 ## File path:

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7547: Add support minor compaction with segment locking

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7547: Add support minor compaction with segment locking URL: https://github.com/apache/incubator-druid/pull/7547#discussion_r298815031 ## File path:

[GitHub] [incubator-druid] clintropolis commented on issue #7998: Binary license management system

2019-06-29 Thread GitBox
clintropolis commented on issue #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998#issuecomment-506998301 Heh, travis rat check failing for `license.yaml` https://travis-ci.org/apache/incubator-druid/jobs/552284036#L644

[GitHub] [incubator-druid] jihoonson opened a new pull request #7998: Binary license management system

2019-06-29 Thread GitBox
jihoonson opened a new pull request #7998: Binary license management system URL: https://github.com/apache/incubator-druid/pull/7998 ### Motivation We are currently maintaining LICENSE.BINARY file manually. The way we do now is 1) Generating dependency reports using maven and

[GitHub] [incubator-druid] clintropolis commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
clintropolis commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298812221 ## File path: .travis.yml ## @@ -51,7 +51,7 @@ matrix: - env: -

[GitHub] [incubator-druid] khwj opened a new pull request #7997: Update Docker build

2019-06-29 Thread GitBox
khwj opened a new pull request #7997: Update Docker build URL: https://github.com/apache/incubator-druid/pull/7997 ### Description - update Docker entrypoint script in respond to the new directory structure #7590 - update Dockerfile to allow dependency caching in multi-stage build

[GitHub] [incubator-druid] khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298806995 ## File path: .travis.yml ## @@ -51,7 +51,7 @@ matrix: - env: -

[GitHub] [incubator-druid] vogievetsky commented on issue #7915: remove deprecated standalone realtime node

2019-06-29 Thread GitBox
vogievetsky commented on issue #7915: remove deprecated standalone realtime node URL: https://github.com/apache/incubator-druid/pull/7915#issuecomment-506973589 Doc updates look great @clintropolis , thank you! This is an

[GitHub] [incubator-druid] fjy commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
fjy commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298804969 ## File path: .travis.yml ## @@ -51,7 +51,7 @@ matrix: - env: -

[GitHub] [incubator-druid] vogievetsky commented on issue #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
vogievetsky commented on issue #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#issuecomment-506970873 Rename checks out  This is an automated message

[GitHub] [incubator-druid] khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298801689 ## File path: extensions-contrib/tdigestsketch/pom.xml ## @@ -29,7 +29,7 @@

[GitHub] [incubator-druid] khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
khwj commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298801689 ## File path: extensions-contrib/tdigestsketch/pom.xml ## @@ -29,7 +29,7 @@

[GitHub] [incubator-druid] gianm merged pull request #7995: Bump commons-codec from 1.7 to 1.12

2019-06-29 Thread GitBox
gianm merged pull request #7995: Bump commons-codec from 1.7 to 1.12 URL: https://github.com/apache/incubator-druid/pull/7995 This is an automated message from the Apache Git Service. To respond to the message, please log on

[incubator-druid] branch master updated: Bump commons-codec from 1.7 to 1.12 (#7995)

2019-06-29 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/incubator-druid.git The following commit(s) were added to refs/heads/master by this push: new 6395c08 Bump commons-codec from 1.7 to

[GitHub] [incubator-druid] gianm commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
gianm commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298801443 ## File path: extensions-contrib/tdigestsketch/pom.xml ## @@ -29,7 +29,7 @@

[GitHub] [incubator-druid] gianm commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
gianm commented on a change in pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996#discussion_r298801427 ## File path: extensions-contrib/tdigestsketch/pom.xml ## @@ -29,7 +29,7 @@

[GitHub] [incubator-druid] khwj opened a new pull request #7996: Rename io.druid to org.druid in tdigestsketch extension

2019-06-29 Thread GitBox
khwj opened a new pull request #7996: Rename io.druid to org.druid in tdigestsketch extension URL: https://github.com/apache/incubator-druid/pull/7996 ### Description For some reason tdigestsketch extension is still using `io.druid` name, probably a small miss from code review. Also

[GitHub] [incubator-druid] asdf2014 opened a new pull request #7995: Bump commons-codec from 1.7 to 1.12

2019-06-29 Thread GitBox
asdf2014 opened a new pull request #7995: Bump commons-codec from 1.7 to 1.12 URL: https://github.com/apache/incubator-druid/pull/7995 Details: [RELEASE NOTES](https://github.com/apache/commons-codec/blob/master/RELEASE-NOTES.txt)

[GitHub] [incubator-druid] leventov commented on a change in pull request #7991: Add development principles

2019-06-29 Thread GitBox
leventov commented on a change in pull request #7991: Add development principles URL: https://github.com/apache/incubator-druid/pull/7991#discussion_r298792734 ## File path: dev/principles.md ## @@ -0,0 +1,1281 @@ +# Druid development principles + +This document enumerates

[GitHub] [incubator-druid] sanastas commented on issue #5698: Oak: New Concurrent Key-Value Map

2019-06-29 Thread GitBox
sanastas commented on issue #5698: Oak: New Concurrent Key-Value Map URL: https://github.com/apache/incubator-druid/issues/5698#issuecomment-506938813 @jon-wei , I couldn't ask for a better start of the conversation! Thank you so much! Totally agree with you, given that