You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@druid.apache.org by Anastasia Braginsky <an...@oath.com.INVALID> on 2018/08/20 09:44:49 UTC

Oak Incremental Index Pull Request

Hi Again,
I am afraid that my previous pull request was hiding behind the previous mail, so not all had a chance to get it :)So here is one more time!We would really appreciate any comments! 

If the request is too big, or if there is any other problem with the request, we would like to hear!We would be happy to assist and make this review happen :)

Here is the latest Pool Request #6184
OakIncrementalIndex has a Plain mode by sanastas · Pull Request #6184 · apache/incubator-druid

| 
| 
| 
| 
 | 
 |

 |

 |
| 
|  | 
OakIncrementalIndex has a Plain mode by sanastas · Pull Request #6184 · ...

Hi Everybody, Please take a look on the following Oak Incremental Index Pool Request. Below please find a list o...
 |

 |

 |





Thanks,Anastasia

Re: Oak Incremental Index Pull Request

Posted by Anastasia Braginsky <an...@oath.com.INVALID>.
Hi Again! :)
First off all big thanks to Jihoon and Slim for agreeing to review the pool request!We have created a more updated pool request (as we didn't get comments on the old) :https://github.com/apache/incubator-druid/pull/6235
And to encourage you to review may be you can find the attached index ingestion benchmark results interesting.Looking forward to hear your comments!!Thanks!
   On Monday, August 20, 2018, 10:46:27 PM GMT+3, Jihoon Son <ji...@apache.org> wrote:  
 
 Hi Anastasia,

thanks for the pull request! I'll happily check it in a few days.

Best,
Jihoon

On Mon, Aug 20, 2018 at 2:44 AM Anastasia Braginsky
<an...@oath.com.invalid> wrote:

> Hi Again,
> I am afraid that my previous pull request was hiding behind the previous
> mail, so not all had a chance to get it :)So here is one more time!We would
> really appreciate any comments!
>
> If the request is too big, or if there is any other problem with the
> request, we would like to hear!We would be happy to assist and make this
> review happen :)
>
> Here is the latest Pool Request #6184
> OakIncrementalIndex has a Plain mode by sanastas · Pull Request #6184 ·
> apache/incubator-druid
>
> |
> |
> |
> |
>  |
>  |
>
>  |
>
>  |
> |
> |  |
> OakIncrementalIndex has a Plain mode by sanastas · Pull Request #6184 · ...
>
> Hi Everybody, Please take a look on the following Oak Incremental Index
> Pool Request. Below please find a list o...
>  |
>
>  |
>
>  |
>
>
>
>
>
> Thanks,Anastasia
>

Re: Oak Incremental Index Pull Request

Posted by Jihoon Son <ji...@apache.org>.
Hi Anastasia,

thanks for the pull request! I'll happily check it in a few days.

Best,
Jihoon

On Mon, Aug 20, 2018 at 2:44 AM Anastasia Braginsky
<an...@oath.com.invalid> wrote:

> Hi Again,
> I am afraid that my previous pull request was hiding behind the previous
> mail, so not all had a chance to get it :)So here is one more time!We would
> really appreciate any comments!
>
> If the request is too big, or if there is any other problem with the
> request, we would like to hear!We would be happy to assist and make this
> review happen :)
>
> Here is the latest Pool Request #6184
> OakIncrementalIndex has a Plain mode by sanastas · Pull Request #6184 ·
> apache/incubator-druid
>
> |
> |
> |
> |
>  |
>  |
>
>  |
>
>  |
> |
> |  |
> OakIncrementalIndex has a Plain mode by sanastas · Pull Request #6184 · ...
>
> Hi Everybody, Please take a look on the following Oak Incremental Index
> Pool Request. Below please find a list o...
>  |
>
>  |
>
>  |
>
>
>
>
>
> Thanks,Anastasia
>