You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by pratik pandey <pr...@gmail.com> on 2018/01/12 15:19:14 UTC
Review Request 65128: ATLAS-1921 : UI : Search using entity and trait
attributes : UI doesn't perform range check and allows providing out
of bounds values for integral and float data types.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65128/
-----------------------------------------------------------
Review request for atlas, keval bhatt, Madhan Neethiraj, Nixon Rodrigues, and Sharmadha Sainath.
Bugs: ATLAS-1921
https://issues.apache.org/jira/browse/ATLAS-1921
Repository: atlas
Description
-------
When applying filter for integral/float data types , Number picker (the up and down button) doesn't do range check according to the data type and lets user go up and down and set any value.Hence when filter is applied and search is made , 500 Internal server error is thrown.
Ex : Maximum value for an integer data type is 2 31 -1
Number picker for integer allows user go to 2147483648 and throws Invalid number exception when searched.
Diffs
-----
dashboardv2/public/css/scss/override.scss d21a697
dashboardv2/public/js/utils/Enums.js 7aca247
dashboardv2/public/js/views/search/QueryBuilderView.js ef84c2f
Diff: https://reviews.apache.org/r/65128/diff/1/
Testing
-------
Done one round of sanity testing with this range :
type:byte size:8 min:-128 max:127
type:short size:16 min:-32768 max:32767
type:char size:16 min:0 max:65535
type:int size:32 min:-2147483648 max:2147483647
type:long size:64 min:-9223372036854775808 max:9223372036854775807
type:float size:32 min:1.4E-45 max:3.4028235E38
type:double size:64 min:4.9E-324 max:1.7976931348623157E308
Thanks,
pratik pandey
Re: Review Request 65128: ATLAS-1921 : UI : Search using entity and
trait
attributes : UI doesn't perform range check and allows providing out of
bounds values for integral and float data types.
Posted by keval bhatt <kb...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65128/#review195387
-----------------------------------------------------------
dashboardv2/public/js/views/search/QueryBuilderView.js
Line 118 (original)
<https://reviews.apache.org/r/65128/#comment274555>
Its better if you make obj.type = "integer" for int,byte,short
- keval bhatt
On Jan. 12, 2018, 3:19 p.m., pratik pandey wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65128/
> -----------------------------------------------------------
>
> (Updated Jan. 12, 2018, 3:19 p.m.)
>
>
> Review request for atlas, keval bhatt, Madhan Neethiraj, Nixon Rodrigues, and Sharmadha Sainath.
>
>
> Bugs: ATLAS-1921
> https://issues.apache.org/jira/browse/ATLAS-1921
>
>
> Repository: atlas
>
>
> Description
> -------
>
> When applying filter for integral/float data types , Number picker (the up and down button) doesn't do range check according to the data type and lets user go up and down and set any value.Hence when filter is applied and search is made , 500 Internal server error is thrown.
>
> Ex : Maximum value for an integer data type is 2 31 -1
> Number picker for integer allows user go to 2147483648 and throws Invalid number exception when searched.
>
>
> Diffs
> -----
>
> dashboardv2/public/css/scss/override.scss d21a697
> dashboardv2/public/js/utils/Enums.js 7aca247
> dashboardv2/public/js/views/search/QueryBuilderView.js ef84c2f
>
>
> Diff: https://reviews.apache.org/r/65128/diff/1/
>
>
> Testing
> -------
>
> Done one round of sanity testing with this range :
> type:byte size:8 min:-128 max:127
> type:short size:16 min:-32768 max:32767
> type:char size:16 min:0 max:65535
> type:int size:32 min:-2147483648 max:2147483647
> type:long size:64 min:-9223372036854775808 max:9223372036854775807
> type:float size:32 min:1.4E-45 max:3.4028235E38
> type:double size:64 min:4.9E-324 max:1.7976931348623157E308
>
>
> Thanks,
>
> pratik pandey
>
>
Re: Review Request 65128: ATLAS-1921 : UI : Search using entity and
trait
attributes : UI doesn't perform range check and allows providing out of
bounds values for integral and float data types.
Posted by Madhan Neethiraj <ma...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65128/#review195391
-----------------------------------------------------------
Ship it!
Ship It!
- Madhan Neethiraj
On Jan. 13, 2018, 8:08 a.m., pratik pandey wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/65128/
> -----------------------------------------------------------
>
> (Updated Jan. 13, 2018, 8:08 a.m.)
>
>
> Review request for atlas, keval bhatt, Madhan Neethiraj, Nixon Rodrigues, and Sharmadha Sainath.
>
>
> Bugs: ATLAS-1921
> https://issues.apache.org/jira/browse/ATLAS-1921
>
>
> Repository: atlas
>
>
> Description
> -------
>
> When applying filter for integral/float data types , Number picker (the up and down button) doesn't do range check according to the data type and lets user go up and down and set any value.Hence when filter is applied and search is made , 500 Internal server error is thrown.
>
> Ex : Maximum value for an integer data type is 2 31 -1
> Number picker for integer allows user go to 2147483648 and throws Invalid number exception when searched.
>
>
> Diffs
> -----
>
> dashboardv2/public/css/scss/override.scss d21a697
> dashboardv2/public/js/utils/Enums.js 7aca247
> dashboardv2/public/js/views/search/QueryBuilderView.js ef84c2f
>
>
> Diff: https://reviews.apache.org/r/65128/diff/2/
>
>
> Testing
> -------
>
> Done one round of sanity testing with this range :
> type:byte size:8 min:-128 max:127
> type:short size:16 min:-32768 max:32767
> type:char size:16 min:0 max:65535
> type:int size:32 min:-2147483648 max:2147483647
> type:long size:64 min:-9223372036854775808 max:9223372036854775807
> type:float size:32 min:1.4E-45 max:3.4028235E38
> type:double size:64 min:4.9E-324 max:1.7976931348623157E308
>
>
> Thanks,
>
> pratik pandey
>
>
Re: Review Request 65128: ATLAS-1921 : UI : Search using entity and
trait
attributes : UI doesn't perform range check and allows providing out of
bounds values for integral and float data types.
Posted by pratik pandey <pr...@gmail.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/65128/
-----------------------------------------------------------
(Updated Jan. 13, 2018, 8:08 a.m.)
Review request for atlas, keval bhatt, Madhan Neethiraj, Nixon Rodrigues, and Sharmadha Sainath.
Changes
-------
review comment handle
Bugs: ATLAS-1921
https://issues.apache.org/jira/browse/ATLAS-1921
Repository: atlas
Description
-------
When applying filter for integral/float data types , Number picker (the up and down button) doesn't do range check according to the data type and lets user go up and down and set any value.Hence when filter is applied and search is made , 500 Internal server error is thrown.
Ex : Maximum value for an integer data type is 2 31 -1
Number picker for integer allows user go to 2147483648 and throws Invalid number exception when searched.
Diffs (updated)
-----
dashboardv2/public/css/scss/override.scss d21a697
dashboardv2/public/js/utils/Enums.js 7aca247
dashboardv2/public/js/views/search/QueryBuilderView.js ef84c2f
Diff: https://reviews.apache.org/r/65128/diff/2/
Changes: https://reviews.apache.org/r/65128/diff/1-2/
Testing
-------
Done one round of sanity testing with this range :
type:byte size:8 min:-128 max:127
type:short size:16 min:-32768 max:32767
type:char size:16 min:0 max:65535
type:int size:32 min:-2147483648 max:2147483647
type:long size:64 min:-9223372036854775808 max:9223372036854775807
type:float size:32 min:1.4E-45 max:3.4028235E38
type:double size:64 min:4.9E-324 max:1.7976931348623157E308
Thanks,
pratik pandey