You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Michael Ho (Code Review)" <ge...@cloudera.org> on 2016/06/30 19:39:25 UTC

[Impala-CR](cdh5-trunk) Revert "IMPALA-1619: Support 64-bit allocations."

Michael Ho has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3543

Change subject: Revert "IMPALA-1619: Support 64-bit allocations."
......................................................................

Revert "IMPALA-1619: Support 64-bit allocations."

This reverts commit 1ffb2bd5a2a2faaa759ebdbaf49bf00aa8f86b5e.

Unbreak the packaging builds for now.

Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
---
M be/src/exec/delimited-text-parser.cc
M be/src/exec/delimited-text-parser.h
M be/src/exec/delimited-text-parser.inline.h
M be/src/exec/hdfs-scanner.cc
M be/src/exec/hdfs-scanner.h
M be/src/exec/hdfs-sequence-scanner.cc
M be/src/exec/hdfs-text-scanner.cc
M be/src/exec/hdfs-text-scanner.h
M be/src/exec/scanner-context.cc
M be/src/runtime/buffered-block-mgr.cc
M be/src/runtime/collection-value-builder.h
M be/src/runtime/free-pool-test.cc
M be/src/runtime/free-pool.h
M be/src/runtime/mem-pool-test.cc
M be/src/runtime/mem-pool.h
M be/src/runtime/string-buffer-test.cc
M be/src/runtime/string-buffer.h
M be/src/udf/udf-internal.h
M be/src/udf/udf.cc
M be/src/udf/udf.h
M be/src/util/bit-util.h
M be/src/util/codec.cc
M be/src/util/codec.h
M be/src/util/decompress-test.cc
M be/src/util/decompress.cc
M common/thrift/generate_error_codes.py
M infra/python/bootstrap_virtualenv.py
D infra/python/deps/python-snappy-0.5.tar.gz
M infra/python/deps/requirements.txt
M tests/query_test/test_compressed_formats.py
30 files changed, 338 insertions(+), 363 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/43/3543/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3543
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <kw...@cloudera.com>

[Impala-CR](cdh5-trunk) Revert "IMPALA-1619: Support 64-bit allocations."

Posted by "Dan Hecht (Code Review)" <ge...@cloudera.org>.
Dan Hecht has posted comments on this change.

Change subject: Revert "IMPALA-1619: Support 64-bit allocations."
......................................................................


Patch Set 1: Code-Review+2

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dh...@cloudera.com>
Gerrit-HasComments: No

[Impala-CR](cdh5-trunk) Revert "IMPALA-1619: Support 64-bit allocations."

Posted by "Michael Ho (Code Review)" <ge...@cloudera.org>.
Michael Ho has posted comments on this change.

Change subject: Revert "IMPALA-1619: Support 64-bit allocations."
......................................................................


Patch Set 1: Verified+1

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dh...@cloudera.com>
Gerrit-Reviewer: Michael Ho <kw...@cloudera.com>
Gerrit-HasComments: No

[Impala-CR](cdh5-trunk) Revert "IMPALA-1619: Support 64-bit allocations."

Posted by "Michael Ho (Code Review)" <ge...@cloudera.org>.
Michael Ho has posted comments on this change.

Change subject: Revert "IMPALA-1619: Support 64-bit allocations."
......................................................................


Patch Set 1:

http://golden.jenkins.cloudera.com/job/generic-package-sles64-11-impala/16074/ is green again.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dh...@cloudera.com>
Gerrit-Reviewer: Michael Ho <kw...@cloudera.com>
Gerrit-HasComments: No

[Impala-CR](cdh5-trunk) Revert "IMPALA-1619: Support 64-bit allocations."

Posted by "Michael Ho (Code Review)" <ge...@cloudera.org>.
Michael Ho has submitted this change and it was merged.

Change subject: Revert "IMPALA-1619: Support 64-bit allocations."
......................................................................


Revert "IMPALA-1619: Support 64-bit allocations."

This reverts commit 1ffb2bd5a2a2faaa759ebdbaf49bf00aa8f86b5e.

Unbreak the packaging builds for now.

Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
Reviewed-on: http://gerrit.cloudera.org:8080/3543
Reviewed-by: Dan Hecht <dh...@cloudera.com>
Tested-by: Michael Ho <kw...@cloudera.com>
---
M be/src/exec/delimited-text-parser.cc
M be/src/exec/delimited-text-parser.h
M be/src/exec/delimited-text-parser.inline.h
M be/src/exec/hdfs-scanner.cc
M be/src/exec/hdfs-scanner.h
M be/src/exec/hdfs-sequence-scanner.cc
M be/src/exec/hdfs-text-scanner.cc
M be/src/exec/hdfs-text-scanner.h
M be/src/exec/scanner-context.cc
M be/src/runtime/buffered-block-mgr.cc
M be/src/runtime/collection-value-builder.h
M be/src/runtime/free-pool-test.cc
M be/src/runtime/free-pool.h
M be/src/runtime/mem-pool-test.cc
M be/src/runtime/mem-pool.h
M be/src/runtime/string-buffer-test.cc
M be/src/runtime/string-buffer.h
M be/src/udf/udf-internal.h
M be/src/udf/udf.cc
M be/src/udf/udf.h
M be/src/util/bit-util.h
M be/src/util/codec.cc
M be/src/util/codec.h
M be/src/util/decompress-test.cc
M be/src/util/decompress.cc
M common/thrift/generate_error_codes.py
M infra/python/bootstrap_virtualenv.py
D infra/python/deps/python-snappy-0.5.tar.gz
M infra/python/deps/requirements.txt
M tests/query_test/test_compressed_formats.py
30 files changed, 338 insertions(+), 363 deletions(-)

Approvals:
  Michael Ho: Verified
  Dan Hecht: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Id079acb83d35b51ba4dfe1c8042e1c5ec891d807
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Michael Ho <kw...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dh...@cloudera.com>
Gerrit-Reviewer: Michael Ho <kw...@cloudera.com>