[Impala-ASF-CR] IMPALA-10173: (Addendum) Fix substitution for unsafe expressions, column-level compatibility check

2023-07-27 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20184 ) Change subject: IMPALA-10173: (Addendum) Fix substitution for unsafe expressions, column-level compatibility check .. Patch Set 4:

[Impala-ASF-CR] IMPALA-12314: Pre-compile LLVM bytecode with Os

2023-07-27 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20265 ) Change subject: IMPALA-12314: Pre-compile LLVM bytecode with Os .. Patch Set 6: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-12269: Codegen cache false negative because of function names hash

2023-07-26 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/20168 ) Change subject: IMPALA-12269: Codegen cache false negative because of function names hash .. IMPALA-12269: Codegen cache false negative

[Impala-ASF-CR] IMPALA-10173: (Addendum) Fix substitution for unsafe expressions, column-level compatibility check

2023-07-26 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20184 ) Change subject: IMPALA-10173: (Addendum) Fix substitution for unsafe expressions, column-level compatibility check .. Patch Set 3:

[Impala-ASF-CR] IMPALA-12292: TestCodegenCache.{test codegen cache with asm module dir,test codegen cache with perf map} fail in builds

2023-07-26 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20224 ) Change subject: IMPALA-12292: TestCodegenCache.{test_codegen_cache_with_asm_module_dir,test_codegen_cache_with_perf_map} fail in builds

[Impala-ASF-CR] IMPALA-12314: Pre-compile LLVM bytecode with Os

2023-07-26 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20265 ) Change subject: IMPALA-12314: Pre-compile LLVM bytecode with Os .. Patch Set 4: (2 comments)

[Impala-ASF-CR] IMPALA-12315: Remove codegen inline hint

2023-07-26 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20266 ) Change subject: IMPALA-12315: Remove codegen inline hint .. Patch Set 1: Code-Review+1 Looks good, thanks. -- To view, visit

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-25 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 8: (3 comments)

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-25 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-24 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 7: (1 comment)

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-24 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12269: Codegen cache false negative because of function names hash

2023-07-24 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/20168 ) Change subject: IMPALA-12269: Codegen cache false negative because of function names hash .. IMPALA-12269: Codegen cache false negative

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-24 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#6). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12292: TestCodegenCache.{test codegen cache with asm module dir,test codegen cache with perf map} fail in builds

2023-07-24 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/20224 ) Change subject: IMPALA-12292: TestCodegenCache.{test_codegen_cache_with_asm_module_dir,test_codegen_cache_with_perf_map} fail in builds

[Impala-ASF-CR] IMPALA-12292: TestCodegenCache.{test codegen cache with asm module dir,test codegen cache with perf map} fail in builds

2023-07-21 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20224 ) Change subject: IMPALA-12292: TestCodegenCache.{test_codegen_cache_with_asm_module_dir,test_codegen_cache_with_perf_map} fail in builds

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-21 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-20 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 4: There are still some TODOs. -- To

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-20 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12292: TestCodegenCache.{test codegen cache with asm module dir,test codegen cache with perf map} fail in builds

2023-07-20 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20224 ) Change subject: IMPALA-12292: TestCodegenCache.{test_codegen_cache_with_asm_module_dir,test_codegen_cache_with_perf_map} fail in builds

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-20 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 3: (2 comments)

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-19 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-19 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 2: (2 comments)

[Impala-ASF-CR] IMPALA-12292: TestCodegenCache.{test codegen cache with asm module dir,test codegen cache with perf map} fail in builds

2023-07-19 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20224 Change subject: IMPALA-12292: TestCodegenCache.{test_codegen_cache_with_asm_module_dir,test_codegen_cache_with_perf_map} fail in builds

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-19 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-17 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 1: (4 comments)

[Impala-ASF-CR] IMPALA-12285: Use targetType in string-to-numeric literal conversion

2023-07-14 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20197 ) Change subject: IMPALA-12285: Use targetType in string-to-numeric literal conversion .. Patch Set 3: Code-Review+2 -- To view,

[Impala-ASF-CR] IMPALA-12285: Use targetType in string-to-numeric literal conversion

2023-07-14 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20197 ) Change subject: IMPALA-12285: Use targetType in string-to-numeric literal conversion .. Patch Set 1: (2 comments)

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-14 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20192 ) Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-12239: BitWidthZeroRepeated seems to be flaky

2023-07-14 Thread Daniel Becker (Code Review)
Daniel Becker has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20187 ) Change subject: IMPALA-12239: BitWidthZeroRepeated seems to be flaky .. IMPALA-12239: BitWidthZeroRepeated seems to be flaky

[Impala-ASF-CR] IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart

2023-07-13 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20192 Change subject: IMPALA-12267: DMLs/DDLs can hang as a result of catalogd restart .. IMPALA-12267: DMLs/DDLs can hang as a result of

[Impala-ASF-CR] IMPALA-12239: BitWidthZeroRepeated seems to be flaky

2023-07-13 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20187 ) Change subject: IMPALA-12239: BitWidthZeroRepeated seems to be flaky .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-12239: BitWidthZeroRepeated seems to be flaky

2023-07-12 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20187 ) Change subject: IMPALA-12239: BitWidthZeroRepeated seems to be flaky .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-12260: Crash if '--asm module dir' is set

2023-07-12 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20155 ) Change subject: IMPALA-12260: Crash if '--asm_module_dir' is set .. Patch Set 4: I ran a perf a-b benchmark:

[Impala-ASF-CR] IMPALA-10173: Fix substitution for unsafe expressions, column-level compatibility check

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20184 ) Change subject: IMPALA-10173: Fix substitution for unsafe expressions, column-level compatibility check .. Patch Set 2: (12

[Impala-ASF-CR] IMPALA-12239: BitWidthZeroRepeated seems to be flaky

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20187 ) Change subject: IMPALA-12239: BitWidthZeroRepeated seems to be flaky .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-12260: Crash if '--asm module dir' is set

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/20155 ) Change subject: IMPALA-12260: Crash if '--asm_module_dir' is set .. IMPALA-12260: Crash if '--asm_module_dir' is set If Impala is

[Impala-ASF-CR] IMPALA-12239: BitWidthZeroRepeated seems to be flaky

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20187 Change subject: IMPALA-12239: BitWidthZeroRepeated seems to be flaky .. IMPALA-12239: BitWidthZeroRepeated seems to be flaky

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#13). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#14). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#15). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-12269: Codegen cache false negative because of function names hash

2023-07-10 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20168 ) Change subject: IMPALA-12269: Codegen cache false negative because of function names hash .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-12269: Codegen cache false negative because of function names hash

2023-07-10 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/20168 ) Change subject: IMPALA-12269: Codegen cache false negative because of function names hash .. IMPALA-12269: Codegen cache false negative

[Impala-ASF-CR] IMPALA-12260: Crash if '--asm module dir' is set

2023-07-07 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20155 ) Change subject: IMPALA-12260: Crash if '--asm_module_dir' is set .. Patch Set 3: (5 comments) Thanks for the reviews.

[Impala-ASF-CR] IMPALA-12260: Crash if '--asm module dir' is set

2023-07-07 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/20155 ) Change subject: IMPALA-12260: Crash if '--asm_module_dir' is set .. IMPALA-12260: Crash if '--asm_module_dir' is set If Impala is

[Impala-ASF-CR] IMPALA-12260: Crash if '--asm module dir' is set

2023-07-07 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#3). ( http://gerrit.cloudera.org:8080/20155 ) Change subject: IMPALA-12260: Crash if '--asm_module_dir' is set .. IMPALA-12260: Crash if '--asm_module_dir' is set If Impala is

[Impala-ASF-CR] IMPALA-12269: Codegen cache false negative because of function names hash

2023-07-07 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/20168 ) Change subject: IMPALA-12269: Codegen cache false negative because of function names hash .. IMPALA-12269: Codegen cache false negative

[Impala-ASF-CR] IMPALA-12269: Codegen cache false negative because of function names hash

2023-07-07 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20168 Change subject: IMPALA-12269: Codegen cache false negative because of function names hash .. IMPALA-12269: Codegen cache false

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-06 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#11). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-04 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#10). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-04 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-07-04 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. Patch Set 9: (7 comments)

[Impala-ASF-CR] IMPALA-12263: Build with C++ Avro library when USE AVRO CPP is true

2023-07-04 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20156 ) Change subject: IMPALA-12263: Build with C++ Avro library when USE_AVRO_CPP is true .. Patch Set 1: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-12263: Build with C++ Avro library when USE AVRO CPP is true

2023-07-03 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20156 ) Change subject: IMPALA-12263: Build with C++ Avro library when USE_AVRO_CPP is true .. Patch Set 1: (1 comment) Looks good, thanks.

[Impala-ASF-CR] IMPALA-12260: Crash if '--asm module dir' is set

2023-07-03 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20155 Change subject: IMPALA-12260: Crash if '--asm_module_dir' is set .. IMPALA-12260: Crash if '--asm_module_dir' is set If Impala is

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-06-30 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table

2023-06-29 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19881 ) Change subject: IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table .. Patch Set 11:

[Impala-ASF-CR] IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted

2023-06-29 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#7). ( http://gerrit.cloudera.org:8080/18999 ) Change subject: IMPALA-10753: Incorrect length when multiple CHAR(N) values are inserted .. IMPALA-10753: Incorrect length when

[Impala-ASF-CR] IMPALA-11949: Make backend AVRO C++ library available

2023-06-29 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20097 ) Change subject: IMPALA-11949: Make backend AVRO C++ library available .. Patch Set 4: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-12238: RandomNestedDataGenerator should take a seed argument

2023-06-28 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20136 ) Change subject: IMPALA-12238: RandomNestedDataGenerator should take a seed argument .. Patch Set 1: Verified+1 -- To view, visit

[Impala-ASF-CR] IMPALA-12238: RandomNestedDataGenerator should take a seed argument

2023-06-28 Thread Daniel Becker (Code Review)
Daniel Becker has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20136 ) Change subject: IMPALA-12238: RandomNestedDataGenerator should take a seed argument .. IMPALA-12238:

[Impala-ASF-CR] IMPALA-12238: RandomNestedDataGenerator should take a seed argument

2023-06-28 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20136 Change subject: IMPALA-12238: RandomNestedDataGenerator should take a seed argument .. IMPALA-12238: RandomNestedDataGenerator should

[Impala-ASF-CR] WIP: Implement reading DELTA BINARY PACKED columns in Parquet

2023-06-28 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20041 ) Change subject: WIP: Implement reading DELTA_BINARY_PACKED columns in Parquet .. Patch Set 3: (15 comments)

[Impala-ASF-CR] IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table

2023-06-27 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19881 ) Change subject: IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table .. Patch Set 8:

[Impala-ASF-CR] IMPALA-11898: Add query options in the profile event if the query failed in planning

2023-06-26 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19517 ) Change subject: IMPALA-11898: Add query options in the profile event if the query failed in planning .. Patch Set 4: (1 comment)

[Impala-ASF-CR] IMPALA-11957: Implement Regression functions: regr slope(), regr intercept() and regr r2()

2023-06-26 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19569 ) Change subject: IMPALA-11957: Implement Regression functions: regr_slope(), regr_intercept() and regr_r2() .. Patch Set 14:

[Impala-ASF-CR] IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table

2023-06-23 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19881 ) Change subject: IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table .. Patch Set 7: (6

[Impala-ASF-CR] WIP: IMPALA-12159: Support ORDER BY for collections of variable length types in select list

2023-06-22 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/20108 Change subject: WIP: IMPALA-12159: Support ORDER BY for collections of variable length types in select list .. WIP: IMPALA-12159:

[Impala-ASF-CR] IMPALA-12148: Bump timeouts for TestAsyncDDLTiming::test ctas()

2023-06-21 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20030 ) Change subject: IMPALA-12148: Bump timeouts for TestAsyncDDLTiming::test_ctas() .. Patch Set 1: Code-Review+1 LGTM -- To view,

[Impala-ASF-CR] IMPALA-11949: Make backend AVRO C++ library available

2023-06-21 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20097 ) Change subject: IMPALA-11949: Make backend AVRO C++ library available .. Patch Set 3: (1 comment)

[Impala-ASF-CR] IMPALA-11957: Implement Regression functions: regr slope(), regr intercept() and regr r2()

2023-06-20 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19569 ) Change subject: IMPALA-11957: Implement Regression functions: regr_slope(), regr_intercept() and regr_r2() .. Patch Set 13: (7

[Impala-ASF-CR] IMPALA-12171: Optimize delimited output

2023-06-20 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19894 ) Change subject: IMPALA-12171: Optimize delimited output .. Patch Set 10: Code-Review+1 Thanks. It's ok for me after fixing the test

[Impala-ASF-CR] IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table

2023-06-16 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19881 ) Change subject: IMPALA-10173: Allow implicit casts between numeric and string types when inserting into table .. Patch Set 5: (32

[Impala-ASF-CR] IMPALA-11459: Use new LLVM Pass Manager

2023-06-15 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/20014 ) Change subject: IMPALA-11459: Use new LLVM Pass Manager .. Patch Set 8: Code-Review+1 Thanks Michael, looks good. -- To view, visit

[Impala-ASF-CR] IMPALA-12114: Pull in fix for THRIFT-5705 and add test

2023-06-02 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19962 ) Change subject: IMPALA-12114: Pull in fix for THRIFT-5705 and add test .. Patch Set 3: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-12142: Decrease default fetch size to 8192 in impala-shell

2023-06-01 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19965 ) Change subject: IMPALA-12142: Decrease default fetch_size to 8192 in impala-shell .. Patch Set 2: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-12142: Decrease default fetch size to 8192 in impala-shell

2023-06-01 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19965 ) Change subject: IMPALA-12142: Decrease default fetch_size to 8192 in impala-shell .. Patch Set 2: Code-Review+1 Thanks, looks good.

[Impala-ASF-CR] IMPALA-12142: Decrease default fetch size to 8192 in impala-shell

2023-06-01 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19965 ) Change subject: IMPALA-12142: Decrease default fetch_size to 8192 in impala-shell .. Patch Set 1: (1 comment)

[Impala-ASF-CR] IMPALA-12114: Pull in fix for THRIFT-5705 and add test

2023-06-01 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19962 ) Change subject: IMPALA-12114: Pull in fix for THRIFT-5705 and add test .. Patch Set 2: (1 comment)

[Impala-ASF-CR] IMPALA-8253: Parquet delta encoding and decoding.

2023-05-22 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/12621 ) Change subject: IMPALA-8253: Parquet delta encoding and decoding. .. Patch Set 20: (7 comments)

[Impala-ASF-CR] IMPALA-8253: Parquet delta encoding and decoding.

2023-05-22 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#20). ( http://gerrit.cloudera.org:8080/12621 ) Change subject: IMPALA-8253: Parquet delta encoding and decoding. .. IMPALA-8253: Parquet delta encoding and decoding. Implemented an

[Impala-ASF-CR] IMPALA-12147: Allow collections of fixed length types as non-passthrough children of unions

2023-05-19 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#2). ( http://gerrit.cloudera.org:8080/19903 ) Change subject: IMPALA-12147: Allow collections of fixed length types as non-passthrough children of unions .. IMPALA-12147: Allow

[Impala-ASF-CR] IMPALA-12147: Allow collections of fixed length types as non-passthrough children of unions

2023-05-18 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded this change for review. ( http://gerrit.cloudera.org:8080/19903 Change subject: IMPALA-12147: Allow collections of fixed length types as non-passthrough children of unions .. IMPALA-12147: Allow

[Impala-ASF-CR] IMPALA-11957: Implement Regression functions: regr slope(), regr intercept() and regr r2()

2023-05-18 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19569 ) Change subject: IMPALA-11957: Implement Regression functions: regr_slope(), regr_intercept() and regr_r2() .. Patch Set 10: (8

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-18 Thread Daniel Becker (Code Review)
Daniel Becker has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-17 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#13). ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER BY for

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-17 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#12). ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER BY for

[Impala-ASF-CR] IMPALA-10173: Allow implicit casts from numerics to STRING when inserting into table

2023-05-16 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19881 ) Change subject: IMPALA-10173: Allow implicit casts from numerics to STRING when inserting into table .. Patch Set 3: (41 comments)

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-15 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#11). ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER BY for

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-15 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#10). ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER BY for

[Impala-ASF-CR] IMPALA-12138: Optimize HS2 result vector allocations

2023-05-15 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19879 ) Change subject: IMPALA-12138: Optimize HS2 result vector allocations .. Patch Set 5: Code-Review+2 -- To view, visit

[Impala-ASF-CR] IMPALA-12138: Optimize HS2 result vector allocations

2023-05-15 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19879 ) Change subject: IMPALA-12138: Optimize HS2 result vector allocations .. Patch Set 3: (3 comments)

[Impala-ASF-CR] IMPALA-11957: Implement Regression functions: regr slope(), regr intercept() and regr r2()

2023-05-12 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19569 ) Change subject: IMPALA-11957: Implement Regression functions: regr_slope(), regr_intercept() and regr_r2() .. Patch Set 9: (7

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-12 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#9). ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER BY for

[Impala-ASF-CR] IMPALA-6433: Add read support for PageHeaderV2

2023-05-12 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19793 ) Change subject: IMPALA-6433: Add read support for PageHeaderV2 .. Patch Set 16: Code-Review+2 (1 comment)

[Impala-ASF-CR] IMPALA-6433: Add read support for PageHeaderV2

2023-05-12 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19793 ) Change subject: IMPALA-6433: Add read support for PageHeaderV2 .. Patch Set 13: -Code-Review (3 comments)

[Impala-ASF-CR] IMPALA-12035: Impala accepts very big numbers but fails to store them correctly

2023-05-11 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19856 ) Change subject: IMPALA-12035: Impala accepts very big numbers but fails to store them correctly .. Patch Set 5: (1 comment)

[Impala-ASF-CR] IMPALA-12035: Impala accepts very big numbers but fails to store them correctly

2023-05-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#5). ( http://gerrit.cloudera.org:8080/19856 ) Change subject: IMPALA-12035: Impala accepts very big numbers but fails to store them correctly .. IMPALA-12035: Impala accepts very

[Impala-ASF-CR] IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list

2023-05-11 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#8). ( http://gerrit.cloudera.org:8080/19660 ) Change subject: IMPALA-12019: Support ORDER BY for arrays of fixed length types in select list .. IMPALA-12019: Support ORDER BY for

[Impala-ASF-CR] IMPALA-12119: rename FORCE INLINE to avoid name collision

2023-05-10 Thread Daniel Becker (Code Review)
Daniel Becker has posted comments on this change. ( http://gerrit.cloudera.org:8080/19839 ) Change subject: IMPALA-12119: rename FORCE_INLINE to avoid name collision .. Patch Set 6: Code-Review+1 -- To view, visit

[Impala-ASF-CR] IMPALA-12035: impala-shell accepts very big numbers but fails to store them correctly

2023-05-10 Thread Daniel Becker (Code Review)
Daniel Becker has uploaded a new patch set (#4). ( http://gerrit.cloudera.org:8080/19856 ) Change subject: IMPALA-12035: impala-shell accepts very big numbers but fails to store them correctly .. IMPALA-12035: impala-shell

<    1   2   3   4   5   6   7   8   9   10   >