Adar Dembo has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/11148 )

Change subject: python: fix non-DECIMAL unit test breakage
......................................................................

python: fix non-DECIMAL unit test breakage

For reference, the table schema is:

  builder.add_column('key').type(kudu.int64).nullable(False)
  builder.add_column('unixtime_micros_val', type_=kudu.unixtime_micros, 
nullable=False)
  if kudu.CLIENT_SUPPORTS_DECIMAL:
    builder.add_column('decimal_val', type_=kudu.decimal, precision=5, scale=2)
  builder.add_column('string_val', type_=kudu.string, 
compression=kudu.COMPRESSION_LZ4, encoding='prefix')
  builder.add_column('bool_val', type_=kudu.bool)
  builder.add_column('double_val', type_=kudu.double)
  builder.add_column('int8_val', type_=kudu.int8)
  builder.add_column('binary_val', type_='binary', 
compression=kudu.COMPRESSION_SNAPPY, encoding='prefix')
  builder.add_column('float_val', type_=kudu.float)
  builder.set_primary_keys(['key', 'unixtime_micros_val'])

Change-Id: Ife6cc90ebf229f876854da6cb3a8857c192a296f
Reviewed-on: http://gerrit.cloudera.org:8080/11148
Reviewed-by: Andrew Wong <aw...@cloudera.com>
Reviewed-by: Grant Henke <granthe...@apache.org>
Tested-by: Kudu Jenkins
---
M python/kudu/tests/test_scanner.py
1 file changed, 6 insertions(+), 6 deletions(-)

Approvals:
  Andrew Wong: Looks good to me, approved
  Grant Henke: Looks good to me, approved
  Kudu Jenkins: Verified

--
To view, visit http://gerrit.cloudera.org:8080/11148
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ife6cc90ebf229f876854da6cb3a8857c192a296f
Gerrit-Change-Number: 11148
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Jordan Birdsell <jtbirds...@apache.org>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to