[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r24274 ## File path: fe/src/main/java/org/apache/doris/clone/TabletSchedulerStat.java ## @@ -0,0

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r24545 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r24730 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246667657 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246669339 ## File path: fe/src/main/java/org/apache/doris/clone/TabletInfo.java ## @@ -0,0 +1,922 @@

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246669370 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246669392 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246669410 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246669732 ## File path: fe/src/main/java/org/apache/doris/clone/TabletScheduler.java ## @@ -0,0 +1,1

[GitHub] morningman opened a new issue #522: Index id in Partition instance is different from index id in OlapTable instance

2019-01-10 Thread GitBox
morningman opened a new issue #522: Index id in Partition instance is different from index id in OlapTable instance URL: https://github.com/apache/incubator-doris/issues/522 I don't know why, but it happened. It can only be recovered using very tricky code modification. Add a issue he

[GitHub] imay commented on a change in pull request #450: Add EsScanNode

2019-01-10 Thread GitBox
imay commented on a change in pull request #450: Add EsScanNode URL: https://github.com/apache/incubator-doris/pull/450#discussion_r246680983 ## File path: be/src/exec/es_scan_node.cpp ## @@ -0,0 +1,668 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// o

[GitHub] imay commented on a change in pull request #450: Add EsScanNode

2019-01-10 Thread GitBox
imay commented on a change in pull request #450: Add EsScanNode URL: https://github.com/apache/incubator-doris/pull/450#discussion_r246684072 ## File path: be/src/exec/es_scan_node.cpp ## @@ -0,0 +1,668 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// o

[GitHub] imay commented on a change in pull request #450: Add EsScanNode

2019-01-10 Thread GitBox
imay commented on a change in pull request #450: Add EsScanNode URL: https://github.com/apache/incubator-doris/pull/450#discussion_r246687168 ## File path: be/src/exec/es_scan_node.cpp ## @@ -0,0 +1,668 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// o

[GitHub] imay commented on a change in pull request #450: Add EsScanNode

2019-01-10 Thread GitBox
imay commented on a change in pull request #450: Add EsScanNode URL: https://github.com/apache/incubator-doris/pull/450#discussion_r246681927 ## File path: be/src/exec/es_scan_node.cpp ## @@ -0,0 +1,668 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// o

[GitHub] chaoyli opened a new pull request #523: Fix inconsistency of three replicas belongs one tablet

2019-01-10 Thread GitBox
chaoyli opened a new pull request #523: Fix inconsistency of three replicas belongs one tablet URL: https://github.com/apache/incubator-doris/pull/523 There is A, B, C replica of one tablet. A has 0 - 10 version. B has 0 - 5, 6, 7, 9, 10 version. 1. B has missed versions, so it clo

[GitHub] morningman opened a new pull request #524: Fix bug that schema change does not set null value correctly

2019-01-10 Thread GitBox
morningman opened a new pull request #524: Fix bug that schema change does not set null value correctly URL: https://github.com/apache/incubator-doris/pull/524 This is an automated message from the Apache Git Service. To res

[GitHub] chaoyli closed pull request #524: Fix bug that schema change does not set null value correctly

2019-01-10 Thread GitBox
chaoyli closed pull request #524: Fix bug that schema change does not set null value correctly URL: https://github.com/apache/incubator-doris/pull/524 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance

[GitHub] imay commented on a change in pull request #523: Fix inconsistency of three replicas belongs to one tablet

2019-01-10 Thread GitBox
imay commented on a change in pull request #523: Fix inconsistency of three replicas belongs to one tablet URL: https://github.com/apache/incubator-doris/pull/523#discussion_r246710398 ## File path: be/src/olap/olap_header.cpp ## @@ -460,6 +460,48 @@ void OLAPHeader::add_d

[GitHub] chaoyli closed pull request #523: Fix inconsistency of three replicas belongs to one tablet

2019-01-10 Thread GitBox
chaoyli closed pull request #523: Fix inconsistency of three replicas belongs to one tablet URL: https://github.com/apache/incubator-doris/pull/523 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance:

[GitHub] chaoyli opened a new pull request #525: Fix inconsistency of three replicas belongs to one tablet (#523)

2019-01-10 Thread GitBox
chaoyli opened a new pull request #525: Fix inconsistency of three replicas belongs to one tablet (#523) URL: https://github.com/apache/incubator-doris/pull/525 There are A, B, C replicas of one tablet. A has 0 - 10 version. B has 0 - 5, 6, 7, 9, 10 version. 1. B has missed version

[GitHub] chaoyli closed pull request #525: Fix inconsistency of three replicas belongs to one tablet (#523)

2019-01-10 Thread GitBox
chaoyli closed pull request #525: Fix inconsistency of three replicas belongs to one tablet (#523) URL: https://github.com/apache/incubator-doris/pull/525 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of proven

[GitHub] imay closed pull request #519: Change the default bdbje sync policy to SYNC

2019-01-10 Thread GitBox
imay closed pull request #519: Change the default bdbje sync policy to SYNC URL: https://github.com/apache/incubator-doris/pull/519 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a fore

[GitHub] morningman commented on a change in pull request #336: Implement new tablet repair and balance framework

2019-01-10 Thread GitBox
morningman commented on a change in pull request #336: Implement new tablet repair and balance framework URL: https://github.com/apache/incubator-doris/pull/336#discussion_r246720010 ## File path: fe/src/main/java/org/apache/doris/clone/TabletChecker.java ## @@ -0,0 +1,449

[GitHub] chaoyli commented on a change in pull request #514: Add rowset reader context builder

2019-01-10 Thread GitBox
chaoyli commented on a change in pull request #514: Add rowset reader context builder URL: https://github.com/apache/incubator-doris/pull/514#discussion_r246723895 ## File path: be/src/olap/rowset/rowset_reader_context_builder.h ## @@ -0,0 +1,154 @@ +// Licensed to the Apa

[GitHub] chaoyli closed pull request #514: Add rowset reader context builder

2019-01-10 Thread GitBox
chaoyli closed pull request #514: Add rowset reader context builder URL: https://github.com/apache/incubator-doris/pull/514 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this is a foreign pull

Re: [VOTE] Release Apache Doris 0.9.0-incubating-rc01

2019-01-10 Thread Dave Fisher
Moving this to the Doris dev list - general@incubator is BCC’d. Regards, Dave > On Jan 10, 2019, at 10:23 AM, Makoto Yui wrote: > > Reed, > > Why not cancel rc1 and create rc2 fixing license header issue that Willem > pointed? > It seems it's better to be fixed. > > It also seems that no one

[GitHub] kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out

2019-01-10 Thread GitBox
kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out URL: https://github.com/apache/incubator-doris/issues/458#issuecomment-453394706 > hi @kangkaisen , Is there any failure log in be? I remember there was not any failure log in be.

[GitHub] kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out

2019-01-10 Thread GitBox
kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out URL: https://github.com/apache/incubator-doris/issues/458#issuecomment-453395944 When I restarted a cluster at 1.9 night , this issue happened again. So I deep into this issue yesterda

[GitHub] imay opened a new pull request #526: Remove useless check, not need lsb_release any more

2019-01-10 Thread GitBox
imay opened a new pull request #526: Remove useless check, not need lsb_release any more URL: https://github.com/apache/incubator-doris/pull/526 This is an automated message from the Apache Git Service. To respond to the mes

[GitHub] kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out

2019-01-10 Thread GitBox
kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out URL: https://github.com/apache/incubator-doris/issues/458#issuecomment-453398715 The FE error Log: ` 2019-01-10 14:00:22,198 WARN 2142 [AgentBatchTask.run():133] task exec error. b

[GitHub] kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out

2019-01-10 Thread GitBox
kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out URL: https://github.com/apache/incubator-doris/issues/458#issuecomment-453399131 So I think when the TCP connection is CLOSE_WAIT status, we should close the old TCP connection and creat

[GitHub] lide-reed closed pull request #526: Remove useless check, not need lsb_release any more

2019-01-10 Thread GitBox
lide-reed closed pull request #526: Remove useless check, not need lsb_release any more URL: https://github.com/apache/incubator-doris/pull/526 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As t

[GitHub] kangkaisen opened a new pull request #527: Add retry for TCP CLOSE_WAIT in AgentBatchTask

2019-01-10 Thread GitBox
kangkaisen opened a new pull request #527: Add retry for TCP CLOSE_WAIT in AgentBatchTask URL: https://github.com/apache/incubator-doris/pull/527 Partially fix [458](https://github.com/apache/incubator-doris/issues/458) This

[GitHub] morningman commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out

2019-01-10 Thread GitBox
morningman commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out URL: https://github.com/apache/incubator-doris/issues/458#issuecomment-453400999 You can check these 2 commits: https://github.com/apache/incubator-doris/commit/b5737ee59a4fbced6ca61b

[GitHub] kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out

2019-01-10 Thread GitBox
kangkaisen commented on issue #458: Create table error: java.net.SocketTimeoutException: Read timed out URL: https://github.com/apache/incubator-doris/issues/458#issuecomment-453403362 > You can check these 2 commits: > [b5737ee](https://github.com/apache/incubator-doris/commit/b5737ee5

[GitHub] kangkaisen closed pull request #527: Add retry for TCP CLOSE_WAIT in AgentBatchTask

2019-01-10 Thread GitBox
kangkaisen closed pull request #527: Add retry for TCP CLOSE_WAIT in AgentBatchTask URL: https://github.com/apache/incubator-doris/pull/527 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed below for the sake of provenance: As this

[GitHub] kangkaisen commented on issue #527: Add retry for TCP CLOSE_WAIT in AgentBatchTask

2019-01-10 Thread GitBox
kangkaisen commented on issue #527: Add retry for TCP CLOSE_WAIT in AgentBatchTask URL: https://github.com/apache/incubator-doris/pull/527#issuecomment-453411410 https://github.com/apache/incubator-doris/pull/408 is duplicated with this PR. https://github.com/apache/incubator-doris

[GitHub] kangkaisen closed issue #245: Colocate Join

2019-01-10 Thread GitBox
kangkaisen closed issue #245: Colocate Join URL: https://github.com/apache/incubator-doris/issues/245 This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL a