You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@carbondata.apache.org by ZhuWilliam <al...@gmail.com> on 2017/04/11 06:48:32 UTC
CarbonLock Exception refractor
<http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/file/n10686/WX20170411-143920%402x.png>
<http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/file/n10686/WX20170411-143942%402x.png>
Carbondata will try to lock table/column/dict when loading data. However
,for now ,it silences the exception and just return false,then tells the
user "Table is locked for updation. Please try after some time" which makes
people confused. There are a lot of reasons which can make locking fail.
Maybe we should refractor here to give
users more specific error .
--
View this message in context: http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/CarbonLock-Exception-refractor-tp10686.html
Sent from the Apache CarbonData Mailing List archive mailing list archive at Nabble.com.
Re: CarbonLock Exception refractor
Posted by Liang Chen <ch...@gmail.com>.
Hi William
+1 for improving CarbonLock Exception.
Do you have any proposed solution for this improvement, and can you raise
one PR for it.
Regards
Liang
Venkata Gollamudi wrote
> Yes, that can be more meaningful, detailing the exception type. Any
> related
> changes are welcome.
>
> On Wed, Apr 12, 2017 at 4:45 PM, QiangCai <
> qiangcai@
> > wrote:
>
>> I agree to refactor code to give detail information.
>>
>>
>>
>> --
>> View this message in context: http://apache-carbondata-
>> mailing-list-archive.1130556.n5.nabble.com/CarbonLock-Exception-refractor-
>> tp10686p10864.html
>> Sent from the Apache CarbonData Mailing List archive mailing list archive
>> at Nabble.com.
>>
--
View this message in context: http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/CarbonLock-Exception-refractor-tp10686p11306.html
Sent from the Apache CarbonData Mailing List archive mailing list archive at Nabble.com.
Re: CarbonLock Exception refractor
Posted by Venkata Gollamudi <g....@gmail.com>.
Yes, that can be more meaningful, detailing the exception type. Any related
changes are welcome.
On Wed, Apr 12, 2017 at 4:45 PM, QiangCai <qi...@qq.com> wrote:
> I agree to refactor code to give detail information.
>
>
>
> --
> View this message in context: http://apache-carbondata-
> mailing-list-archive.1130556.n5.nabble.com/CarbonLock-Exception-refractor-
> tp10686p10864.html
> Sent from the Apache CarbonData Mailing List archive mailing list archive
> at Nabble.com.
>
Re: CarbonLock Exception refractor
Posted by QiangCai <qi...@qq.com>.
I agree to refactor code to give detail information.
--
View this message in context: http://apache-carbondata-mailing-list-archive.1130556.n5.nabble.com/CarbonLock-Exception-refractor-tp10686p10864.html
Sent from the Apache CarbonData Mailing List archive mailing list archive at Nabble.com.