[GitHub] [incubator-doris] morningman merged pull request #1510: Fix rollup bug when init RowCursor in MergeContext

2019-07-18 Thread GitBox
morningman merged pull request #1510: Fix rollup bug when init RowCursor in MergeContext URL: https://github.com/apache/incubator-doris/pull/1510 This is an automated message from the Apache Git Service. To respond to the me

[GitHub] [incubator-doris] chenhao7253886 opened a new issue #1512: EXPLAIN info does't include item whether the query hit the table's prefix index.

2019-07-18 Thread GitBox
chenhao7253886 opened a new issue #1512: EXPLAIN info does't include item whether the query hit the table's prefix index. URL: https://github.com/apache/incubator-doris/issues/1512 Add item to EXPLAIN for indexing whether the query hit the table's prefix index. ---

[GitHub] [incubator-doris] zhuyuncheng edited a comment on issue #1509: This code whether only limit operations are supported and offset operations are not supported?

2019-07-18 Thread GitBox
zhuyuncheng edited a comment on issue #1509: This code whether only limit operations are supported and offset operations are not supported? URL: https://github.com/apache/incubator-doris/issues/1509#issuecomment-513103898 > We support both limit and offset. When there is an order by clause

[GitHub] [incubator-doris] WingsGo opened a new issue #1511: UDAF variable parameters support bug

2019-07-18 Thread GitBox
WingsGo opened a new issue #1511: UDAF variable parameters support bug URL: https://github.com/apache/incubator-doris/issues/1511 **Describe the bug** UDAF cannot identify variable parameters, **To Reproduce** if you write a udaf function with variable parameters such as

[GitHub] [incubator-doris] zhuyuncheng commented on issue #1509: This code whether only limit operations are supported and offset operations are not supported?

2019-07-18 Thread GitBox
zhuyuncheng commented on issue #1509: This code whether only limit operations are supported and offset operations are not supported? URL: https://github.com/apache/incubator-doris/issues/1509#issuecomment-513103898 > We support both limit and offset. When there is an order by clause, offse

[GitHub] [incubator-doris] morningman commented on issue #1510: Fix rollup bug when init RowCursor in MergeContext

2019-07-18 Thread GitBox
morningman commented on issue #1510: Fix rollup bug when init RowCursor in MergeContext URL: https://github.com/apache/incubator-doris/pull/1510#issuecomment-513100015 Related to #1502 This is an automated message from the A

[GitHub] [incubator-doris] chaoyli opened a new pull request #1510: Fix rollup bug when init RowCursor in MergeContext

2019-07-18 Thread GitBox
chaoyli opened a new pull request #1510: Fix rollup bug when init RowCursor in MergeContext URL: https://github.com/apache/incubator-doris/pull/1510 When doing rollup, seek_columns equals to the complete set of tablet's columns. There is no necessity to set it.

[GitHub] [incubator-doris] imay commented on issue #1509: This code whether only limit operations are supported and offset operations are not supported?

2019-07-18 Thread GitBox
imay commented on issue #1509: This code whether only limit operations are supported and offset operations are not supported? URL: https://github.com/apache/incubator-doris/issues/1509#issuecomment-513088212 We support both limit and offset. When there is an order by clause, offset will e

[GitHub] [incubator-doris] zhuyuncheng opened a new issue #1509: This code whether only limit operations are supported and offset operations are not supported?

2019-07-18 Thread GitBox
zhuyuncheng opened a new issue #1509: This code whether only limit operations are supported and offset operations are not supported? URL: https://github.com/apache/incubator-doris/issues/1509 🌹🌹 This is an automated message f

[GitHub] [incubator-doris] EmmyMiao87 commented on issue #1488: Add administrator guide of load

2019-07-18 Thread GitBox
EmmyMiao87 commented on issue #1488: Add administrator guide of load URL: https://github.com/apache/incubator-doris/pull/1488#issuecomment-513071715 1. 总文档中隐去transaction 的概念,用户不好理解。 2. 异步导入方式下,用户可以提交的最大job数量和同时能够running的job数量之间的关系,以及对应的两个配置要说明下(你只说了 desired\_max\_waiting\_jobs 这一个) 3

[GitHub] [incubator-doris] imay merged pull request #1507: Fix bug that BE may crash when closing OlapTableSink

2019-07-18 Thread GitBox
imay merged pull request #1507: Fix bug that BE may crash when closing OlapTableSink URL: https://github.com/apache/incubator-doris/pull/1507 This is an automated message from the Apache Git Service. To respond to the messag

[GitHub] [incubator-doris] imay commented on a change in pull request #1505: Add timediff function

2019-07-18 Thread GitBox
imay commented on a change in pull request #1505: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1505#discussion_r305178263 ## File path: fe/src/main/java/org/apache/doris/analysis/IntLiteral.java ## @@ -89,6 +91,11 @@ public IntLiteral(long longV

[GitHub] [incubator-doris] imay commented on a change in pull request #1505: Add timediff function

2019-07-18 Thread GitBox
imay commented on a change in pull request #1505: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1505#discussion_r305177722 ## File path: gensrc/script/gen_functions.py ## @@ -552,6 +554,7 @@ 'VARCHAR': 'string_val', 'DATE': 'datetime_val',

[GitHub] [incubator-doris] imay commented on a change in pull request #1505: Add timediff function

2019-07-18 Thread GitBox
imay commented on a change in pull request #1505: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1505#discussion_r305179681 ## File path: be/src/runtime/result_writer.cpp ## @@ -105,7 +105,8 @@ Status ResultWriter::add_one_row(TupleRow* row) {

[GitHub] [incubator-doris] morningman merged pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-18 Thread GitBox
morningman merged pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-doris] morningman opened a new issue #1508: Use different ID instead of table ID for base index of an OLAP table.

2019-07-18 Thread GitBox
morningman opened a new issue #1508: Use different ID instead of table ID for base index of an OLAP table. URL: https://github.com/apache/incubator-doris/issues/1508 In current implementation, the id of a base index of a OLAP table use the same id as table's id. Because in origin design

[GitHub] [incubator-doris] morningman opened a new pull request #1507: Fix bug that BE may crash when closing OlapTableSink

2019-07-18 Thread GitBox
morningman opened a new pull request #1507: Fix bug that BE may crash when closing OlapTableSink URL: https://github.com/apache/incubator-doris/pull/1507 The `_profile` in OlapTableSink may not be initialized if `prepare()` method is not called. So when close the OlapTableSink, we should

[GitHub] [incubator-doris] morningman opened a new issue #1506: BE crash when closing OlapTableSink

2019-07-18 Thread GitBox
morningman opened a new issue #1506: BE crash when closing OlapTableSink URL: https://github.com/apache/incubator-doris/issues/1506 **Describe the bug** ``` Status OlapTableSink::close(RuntimeState* state, Status close_status) { SCOPED_TIMER(_profile->total_time_counter());

[GitHub] [incubator-doris] morningman commented on issue #1501: reabalance may cause one tablet's 3 replicas scheduled to one machine

2019-07-18 Thread GitBox
morningman commented on issue #1501: reabalance may cause one tablet's 3 replicas scheduled to one machine URL: https://github.com/apache/incubator-doris/issues/1501#issuecomment-513050765 I will fix this bug This is an aut

[GitHub] [incubator-doris] morningman opened a new pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-18 Thread GitBox
morningman opened a new pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489 When query result reach limit, the Coordinator in FE will send a cancel request to BE to cancel the query. And when bei

[GitHub] [incubator-doris] morningman closed pull request #1489: Remove query status report from BE when query is cancelled normally

2019-07-18 Thread GitBox
morningman closed pull request #1489: Remove query status report from BE when query is cancelled normally URL: https://github.com/apache/incubator-doris/pull/1489 This is an automated message from the Apache Git Service. To

[GitHub] [incubator-doris] imay merged pull request #1504: Fix the duplicated request bug of mini load

2019-07-18 Thread GitBox
imay merged pull request #1504: Fix the duplicated request bug of mini load URL: https://github.com/apache/incubator-doris/pull/1504 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-doris] chenhao7253886 commented on issue #1505: Add timediff function

2019-07-18 Thread GitBox
chenhao7253886 commented on issue #1505: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1505#issuecomment-512846057 I think that you need't import a new udf type to implement timediff. This is an aut

[GitHub] [incubator-doris] chenhao7253886 commented on issue #1500: Add timediff function

2019-07-18 Thread GitBox
chenhao7253886 commented on issue #1500: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1500#issuecomment-512841713 If you only want to add timediff function, you need't import a new udf type. This i

[GitHub] [incubator-doris] HangyuanLiu opened a new pull request #1505: Add timediff function

2019-07-18 Thread GitBox
HangyuanLiu opened a new pull request #1505: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1505 Add timediff function This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-doris] chenhao7253886 merged pull request #1503: Fix bug that single partition table get wrong partition type

2019-07-18 Thread GitBox
chenhao7253886 merged pull request #1503: Fix bug that single partition table get wrong partition type URL: https://github.com/apache/incubator-doris/pull/1503 This is an automated message from the Apache Git Service. To res

[GitHub] [incubator-doris] EmmyMiao87 opened a new pull request #1504: Fix the duplicated request bug of mini load

2019-07-18 Thread GitBox
EmmyMiao87 opened a new pull request #1504: Fix the duplicated request bug of mini load URL: https://github.com/apache/incubator-doris/pull/1504 The function of miniLoadBegin will return the txn_id. If the backend sends the duplicated request to frontend, frontend will return the txn_id

[GitHub] [incubator-doris] imay merged pull request #1502: Fix rollup bug when init RowCursor

2019-07-18 Thread GitBox
imay merged pull request #1502: Fix rollup bug when init RowCursor URL: https://github.com/apache/incubator-doris/pull/1502 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-doris] chenhao7253886 opened a new pull request #1503: Fix bug that single partition table get wrong partition type

2019-07-18 Thread GitBox
chenhao7253886 opened a new pull request #1503: Fix bug that single partition table get wrong partition type URL: https://github.com/apache/incubator-doris/pull/1503 This is an automated message from the Apache Git Service.

[GitHub] [incubator-doris] chaoyli opened a new pull request #1502: Fix rollup bug when init RowCursor

2019-07-18 Thread GitBox
chaoyli opened a new pull request #1502: Fix rollup bug when init RowCursor URL: https://github.com/apache/incubator-doris/pull/1502 When doing rollup, seek_columns equals to the complete set of tablet's columns. There is no necessity to set it. -

[GitHub] [incubator-doris] HangyuanLiu closed pull request #1500: Add timediff function

2019-07-18 Thread GitBox
HangyuanLiu closed pull request #1500: Add timediff function URL: https://github.com/apache/incubator-doris/pull/1500 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu