[Discussion] Propose to upgrade the version of integration/presto from 0.187 to 0.206

[Liang Chen]

Hi Dev

The presto community already released 0.206 last week (refer the detail at
https://prestodb.io/docs/current/release/release-0.206.html),  this release
fixed many issues, so propose Apache CarbonData community to upgrade to the
latest presto version for carbondata integration.

please provide your opinion.

Regards
Liang

[bhavya411]

Hi Dev,

Yes, we should definitely go for the 0.206 upgrade for Presto as we are now
using the dictionary_aggregation feature for optimization. The other bug
fixes are also important for carbondata integration.
However, they have changed the connector interface as well, so we might
need to change our interface accordingly.

Thanks and regards
Bhavya

On Tue, Jul 24, 2018 at 2:11 PM, Liang Chen <[hidden email]> wrote:

> Hi Dev
>
> The presto community already released 0.206 last week (refer the detail at
> https://prestodb.io/docs/current/release/release-0.206.html),  this
> release
> fixed many issues, so propose Apache CarbonData community to upgrade to the
> latest presto version for carbondata integration.
>
> please provide your opinion.
>
> Regards
> Liang
>

--
*Bhavya Aggarwal*
CTO & Partner
Knoldus Inc. <http://www.knoldus.com/>
[phone]
Canada - USA - India - Singapore
<https://in.linkedin.com/company/knoldus> <https://twitter.com/Knolspeak>
<https://www.facebook.com/KnoldusSoftware/> <https://blog.knoldus.com/>

[Ajantha Bhat]

Hi Dev,

+1  for the 0.206 upgrade of Presto.
Currently, while running some test (TPCH) queries on 500GB data, we
observed some memory issues in the presto cluster.
Couldn't narrow down the exact memory problem in the presto. We restart
cluster as a workaround.
Hoping that issue is fixed in latest release.

Thanks and regards
Ajantha

On Wed, Jul 25, 2018 at 10:21 AM Bhavya Aggarwal <[hidden email]> wrote:

> Hi Dev,
>
> Yes, we should definitely go for the 0.206 upgrade for Presto as we are now
> using the dictionary_aggregation feature for optimization. The other bug
> fixes are also important for carbondata integration.
> However, they have changed the connector interface as well, so we might
> need to change our interface accordingly.
>
> Thanks and regards
> Bhavya
>
> On Tue, Jul 24, 2018 at 2:11 PM, Liang Chen <[hidden email]>
> wrote:
>
> > Hi Dev
> >
> > The presto community already released 0.206 last week (refer the detail
> at
> > https://prestodb.io/docs/current/release/release-0.206.html),  this
> > release
> > fixed many issues, so propose Apache CarbonData community to upgrade to
> the
> > latest presto version for carbondata integration.
> >
> > please provide your opinion.
> >
> > Regards
> > Liang
> >
>
>
>
> --
> *Bhavya Aggarwal*
> CTO & Partner
> Knoldus Inc. <http://www.knoldus.com/>
> [phone]
> Canada - USA - India - Singapore
> <https://in.linkedin.com/company/knoldus> <https://twitter.com/Knolspeak>
> <https://www.facebook.com/KnoldusSoftware/> <https://blog.knoldus.com/>
>

[Jacky Li]

+1

> 在 2018年7月25日，下午1:14，Ajantha Bhat <[hidden email]> 写道：
>
> Hi Dev,
>
> +1  for the 0.206 upgrade of Presto.
> Currently, while running some test (TPCH) queries on 500GB data, we
> observed some memory issues in the presto cluster.
> Couldn't narrow down the exact memory problem in the presto. We restart
> cluster as a workaround.
> Hoping that issue is fixed in latest release.
>
> Thanks and regards
> Ajantha
>
> On Wed, Jul 25, 2018 at 10:21 AM Bhavya Aggarwal <[hidden email]> wrote:
>
>> Hi Dev,
>>
>> Yes, we should definitely go for the 0.206 upgrade for Presto as we are now
>> using the dictionary_aggregation feature for optimization. The other bug
>> fixes are also important for carbondata integration.
>> However, they have changed the connector interface as well, so we might
>> need to change our interface accordingly.
>>
>> Thanks and regards
>> Bhavya
>>
>> On Tue, Jul 24, 2018 at 2:11 PM, Liang Chen <[hidden email]>
>> wrote:
>>
>>> Hi Dev
>>>
>>> The presto community already released 0.206 last week (refer the detail
>> at
>>> https://prestodb.io/docs/current/release/release-0.206.html),  this
>>> release
>>> fixed many issues, so propose Apache CarbonData community to upgrade to
>> the
>>> latest presto version for carbondata integration.
>>>
>>> please provide your opinion.
>>>
>>> Regards
>>> Liang
>>>
>>
>>
>>
>> --
>> *Bhavya Aggarwal*
>> CTO & Partner
>> Knoldus Inc. <http://www.knoldus.com/>
>> [phone]
>> Canada - USA - India - Singapore
>> <https://in.linkedin.com/company/knoldus> <https://twitter.com/Knolspeak>
>> <https://www.facebook.com/KnoldusSoftware/> <https://blog.knoldus.com/>
>>
>

[kumarvishal09]

+1

-Regards
Kumar Vishal

On Wed, Jul 25, 2018 at 12:01 PM Jacky Li <[hidden email]> wrote:

> +1
>
> > 在 2018年7月25日，下午1:14，Ajantha Bhat <[hidden email]> 写道：
> >
> > Hi Dev,
> >
> > +1  for the 0.206 upgrade of Presto.
> > Currently, while running some test (TPCH) queries on 500GB data, we
> > observed some memory issues in the presto cluster.
> > Couldn't narrow down the exact memory problem in the presto. We restart
> > cluster as a workaround.
> > Hoping that issue is fixed in latest release.
> >
> > Thanks and regards
> > Ajantha
> >
> > On Wed, Jul 25, 2018 at 10:21 AM Bhavya Aggarwal <[hidden email]>
> wrote:
> >
> >> Hi Dev,
> >>
> >> Yes, we should definitely go for the 0.206 upgrade for Presto as we are
> now
> >> using the dictionary_aggregation feature for optimization. The other bug
> >> fixes are also important for carbondata integration.
> >> However, they have changed the connector interface as well, so we might
> >> need to change our interface accordingly.
> >>
> >> Thanks and regards
> >> Bhavya
> >>
> >> On Tue, Jul 24, 2018 at 2:11 PM, Liang Chen <[hidden email]>
> >> wrote:
> >>
> >>> Hi Dev
> >>>
> >>> The presto community already released 0.206 last week (refer the detail
> >> at
> >>> https://prestodb.io/docs/current/release/release-0.206.html),  this
> >>> release
> >>> fixed many issues, so propose Apache CarbonData community to upgrade to
> >> the
> >>> latest presto version for carbondata integration.
> >>>
> >>> please provide your opinion.
> >>>
> >>> Regards
> >>> Liang
> >>>
> >>
> >>
> >>
> >> --
> >> *Bhavya Aggarwal*
> >> CTO & Partner
> >> Knoldus Inc. <http://www.knoldus.com/>
> >> [phone]
> >> Canada - USA - India - Singapore
> >> <https://in.linkedin.com/company/knoldus> <
> https://twitter.com/Knolspeak>
> >> <https://www.facebook.com/KnoldusSoftware/> <https://blog.knoldus.com/>
> >>
> >
>
>

[Liang Chen]

Hi

Just i checked 0.207 and 0.208, there are fixed many memory issues, so
propose to upgrade to 0.208 for Apache CarbonData 1.5.0 integration.

Regards
Liang


bhavya411 wrote

> Hi Dev,
>
> Yes, we should definitely go for the 0.206 upgrade for Presto as we are
> now
> using the dictionary_aggregation feature for optimization. The other bug
> fixes are also important for carbondata integration.
> However, they have changed the connector interface as well, so we might
> need to change our interface accordingly.
>
> Thanks and regards
> Bhavya
>
> On Tue, Jul 24, 2018 at 2:11 PM, Liang Chen &lt;

> chenliang6136@

> &gt; wrote:
>
>> Hi Dev
>>
>> The presto community already released 0.206 last week (refer the detail
>> at
>> https://prestodb.io/docs/current/release/release-0.206.html),  this
>> release
>> fixed many issues, so propose Apache CarbonData community to upgrade to
>> the
>> latest presto version for carbondata integration.
>>
>> please provide your opinion.
>>
>> Regards
>> Liang
>>
>
>
>
> --
> *Bhavya Aggarwal*
> CTO & Partner
> Knoldus Inc. &lt;http://www.knoldus.com/&gt;
> [phone]
> Canada - USA - India - Singapore
> &lt;https://in.linkedin.com/company/knoldus&gt;
> &lt;https://twitter.com/Knolspeak&gt;
> &lt;https://www.facebook.com/KnoldusSoftware/&gt;
> &lt;https://blog.knoldus.com/&gt;

--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/

[sujith chacko]

+1 for presto 0.207/0.208 upgradation.
 As liang mentioned there are some memory issues related patches as well
there  are some security fixes in 0.207 version
Hope the patches are present in presto 0.208 version also.

Thanks,
Sujith

On Mon, 13 Aug 2018 at 2:38 PM, Liang Chen <[hidden email]> wrote:

> Hi
>
> Just i checked 0.207 and 0.208, there are fixed many memory issues, so
> propose to upgrade to 0.208 for Apache CarbonData 1.5.0 integration.
>
> Regards
> Liang
>
>
> bhavya411 wrote
> > Hi Dev,
> >
> > Yes, we should definitely go for the 0.206 upgrade for Presto as we are
> > now
> > using the dictionary_aggregation feature for optimization. The other bug
> > fixes are also important for carbondata integration.
> > However, they have changed the connector interface as well, so we might
> > need to change our interface accordingly.
> >
> > Thanks and regards
> > Bhavya
> >
> > On Tue, Jul 24, 2018 at 2:11 PM, Liang Chen &lt;
>
> > chenliang6136@
>
> > &gt; wrote:
> >
> >> Hi Dev
> >>
> >> The presto community already released 0.206 last week (refer the detail
> >> at
> >> https://prestodb.io/docs/current/release/release-0.206.html),  this
> >> release
> >> fixed many issues, so propose Apache CarbonData community to upgrade to
> >> the
> >> latest presto version for carbondata integration.
> >>
> >> please provide your opinion.
> >>
> >> Regards
> >> Liang
> >>
> >
> >
> >
> > --
> > *Bhavya Aggarwal*
> > CTO & Partner
> > Knoldus Inc. &lt;http://www.knoldus.com/&gt;
> > [phone]
> > Canada - USA - India - Singapore
> > &lt;https://in.linkedin.com/company/knoldus&gt;
> > &lt;https://twitter.com/Knolspeak&gt;
> > &lt;https://www.facebook.com/KnoldusSoftware/&gt;
> > &lt;https://blog.knoldus.com/&gt;
>
>
>
>
>
> --
> Sent from:
> http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/
>

[xm_zzc]

Hi:
  I think we can upgrade presto to 0.210 for CarbonData 1.5, 0.210 has fixed
an 'JDBC Driver' issue which is used frequently:
  *Deallocate prepared statement when PreparedStatement is closed.
Previously, Connection became unusable after many prepared statements were
created.*




--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/

[Liang Chen]

Hi

Agree with upgrading to 0.210, i will work on this pr.

Regards
Liang


xm_zzc wrote
> Hi:
>   I think we can upgrade presto to 0.210 for CarbonData 1.5, 0.210 has
> fixed
> an 'JDBC Driver' issue which is used frequently:
>   *Deallocate prepared statement when PreparedStatement is closed.
> Previously, Connection became unusable after many prepared statements were
> created.*
>
>
>
>
> --
> Sent from:
> http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/





--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/

[Liang Chen]

HI

presto code as below, requires higher java version:
-------------------------
JavaVersion version = JavaVersion.parse(javaVersion);
        if (version.getMajor() == 8 && version.getUpdate().isPresent() &&
version.getUpdate().getAsInt() >= 151) {
            return;
        }

Regards
Liang


xm_zzc wrote

> Hi:
>   I think we can upgrade presto to 0.210 for CarbonData 1.5, 0.210 has
> fixed
> an 'JDBC Driver' issue which is used frequently:
>   *Deallocate prepared statement when PreparedStatement is closed.
> Previously, Connection became unusable after many prepared statements were
> created.*
>
>
>
>
> --
> Sent from:
> http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/

--
Sent from: http://apache-carbondata-dev-mailing-list-archive.1130556.n5.nabble.com/