[GitHub] incubator-distributedlog pull request #118: DL-189 : remove the extra NOTICE...

2017-01-27 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/118

DL-189 : remove the extra NOTICE change



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog DL-189

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/118.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #118


commit 036c894d4ce32fdcf531ce2c995646794b0f24d6
Author: xieliang <xieliang...@gmail.com>
Date:   2017-01-27T09:09:43Z

remove the extra NOTICE change




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #117: DL-187 : load test doc typo fix

2017-01-11 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/117

DL-187 : load test doc typo fix



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-187-doc_typo

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/117.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #117


commit 28d883ef6bd4bf101410939d12a6daabb2de101a
Author: xieliang <xieliang...@gmail.com>
Date:   2017-01-12T07:02:38Z

load test doc typo fix




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #115: DL-184 : reduce server_graceful_...

2017-01-11 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/115

DL-184 : reduce server_graceful_shutdown_period_ms from 35s to 20s, which 
is smaller than the default daemon stop timeout value



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-184-graceful

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/115.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #115






---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #114: DL-186 : using distributedlog-se...

2017-01-10 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/114

DL-186 : using distributedlog-service/bin/dlog, not 
distributedlog-core/bin/dlog; adding more echos to let the end user know the 
detail info



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-186-smoketest

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/114.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #114






---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #112: DL-182 : assembled package shoul...

2017-01-10 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/112

DL-182 : assembled package should include README.md



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-182-assembled

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/112.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #112


commit c97b8b802a0462b8a90dbfcf388ccecb074629cd
Author: xieliang <xieliang...@gmail.com>
Date:   2017-01-10T09:56:08Z

assembled package should include README.md




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #110: DL-181 : avoid starting failure ...

2017-01-09 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/110

DL-181 : avoid starting failure due to call Optional.get() on an absent 
value

see original JIRA for the detailed error msg

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-181-startingProxyFailed

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/110.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #110


commit ca27983b0c2fb6056f87311766e048325c1fc09a
Author: xieliang <xieliang...@gmail.com>
Date:   2017-01-09T10:33:08Z

avoid starting failure due to call Optional.get() on an absent value




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #98: DL-171 : adding a short sleep to ...

2017-01-05 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/98

DL-171 : adding a short sleep to let the WriteCompleteListener have time to 
run before the final position be requested

once the "writer.write" is done, if "writer.position()" be invoked easier 
than the WriteCompleteListener onSuccess callback, due to the "synchronized", 
the position result will be 0, not the expected 33. we can just add a short 
sleep to avoid this test issue.

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-171-Fix-TestAppendOnlyStreamWriter

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/98.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #98


commit 9de4d177d2c4d41aaaeaa69012be01fa4e0104ad
Author: xieliang <xieliang...@gmail.com>
Date:   2017-01-05T08:56:37Z

adding a short sleep to let the WriteCompleteListener have time to run 
before the final position be requested




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #94: DL-167 : fix testCalculateUnequal...

2016-12-30 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/94

DL-167 : fix testCalculateUnequalWeight computing corner case



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog 
DL-167-fix-testCalculateUnequalWeight

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/94.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #94


commit 09447cada3acfb3c9cb02012cab945fbd8c8dc5c
Author: xieliang <xieliang...@gmail.com>
Date:   2016-12-30T08:02:08Z

fix testCalculateUnequalWeight computing corner case




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog issue #91: DL-165 : Add TestTimedOutTestsListener t...

2016-12-29 Thread xieliang
Github user xieliang commented on the issue:

https://github.com/apache/incubator-distributedlog/pull/91
  
what's wrong?  i can build successful on my local box...


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #88: DL-155 : fix flaky TestAsyncReade...

2016-12-22 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/88

DL-155 : fix flaky TestAsyncReaderLock#testReaderLockMultiReadersScenario

Ran the case with changed patch 30 times, all are passed;   and it'll fail 
w/o patch

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog DL-155

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/88.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #88


commit 44d273afde6d04aa35464170bec770b578c92e4b
Author: xieliang <xieliang...@gmail.com>
Date:   2016-12-22T08:46:25Z

fix flaky TestAsyncReaderLock#testReaderLockMultiReadersScenario




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog issue #78: DL-145 : the write requests should be er...

2016-12-20 Thread xieliang
Github user xieliang commented on the issue:

https://github.com/apache/incubator-distributedlog/pull/78
  
The failure cases are not related with current change, please see 
https://issues.apache.org/jira/browse/DL-147 for further working on it.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #79: DL-146 : fix Stream typo : initia...

2016-12-19 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/79

DL-146 : fix Stream typo : initiaize -> initialize



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog DL-146

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/79.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #79


commit 7f7e86af064d81a338a5db4215eeb24afb0ada34
Author: xieliang <xieliang...@gmail.com>
Date:   2016-12-20T07:41:01Z

fix Stream typo : initiaize -> initialize




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #78: DL-145 : the write requests shoul...

2016-12-19 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/78

DL-145 : the write requests should be error out immediately even if the 
rolling writer still be creating

Passed all test cases locally, now 
TestDistributedLogService#testServiceTimeout case is stable on my box

You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog DL-145

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/78.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #78


commit 6be8ca4d01c4c40947d4b901f0299c8dcc97c509
Author: xieliang <xieliang...@gmail.com>
Date:   2016-12-20T07:19:38Z

the write requests should be error out immediately even if the rolling 
writer still be creating




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #73: DL-28 : using /* */ style comment...

2016-12-14 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/73

DL-28 : using /* */ style comment to avoid scrooge windows platform issue



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog DL-28

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/73.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #73


commit df5a66cb57e7d99e1df6e1cf29620876cd6827c6
Author: xieliang <xieliang...@gmail.com>
Date:   2016-12-15T03:43:46Z

using /* */ style comment to avoid scrooge windows platform issue




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread xieliang
Github user xieliang commented on the issue:

https://github.com/apache/incubator-distributedlog/pull/71
  
It's not the same thing. The travis ci runs with "findbugs-maven-plugin", 
and i ran with findbugs binary package. 
(From http://gleclaire.github.io/findbugs-maven-plugin/index.html , i see 
the core findbugs version are all 3.0.1.)
Let me finger out the diff later


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog issue #71: DL-140 : Fix distributedlog-core findbug...

2016-12-13 Thread xieliang
Github user xieliang commented on the issue:

https://github.com/apache/incubator-distributedlog/pull/71
  
I ran the findbug (3.0.1) binary on my local devbox. Those multi-thread 
warings exist even after importing the findbug exclude xml file manually,  
what's the findbugs binary version at ci side, i guess it's obsolete...


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #71: DL-140 : Fix distributedlog-core ...

2016-12-13 Thread xieliang
GitHub user xieliang opened a pull request:

https://github.com/apache/incubator-distributedlog/pull/71

DL-140 : Fix distributedlog-core findbug inconsistent synchronization 
warings



You can merge this pull request into a Git repository by running:

$ git pull https://github.com/xieliang/incubator-distributedlog DL-140

Alternatively you can review and apply these changes as the patch at:

https://github.com/apache/incubator-distributedlog/pull/71.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

This closes #71


commit febd945ffa409d861d0f783a78831b88ae56c540
Author: xieliang <xieliang...@gmail.com>
Date:   2016-12-13T10:11:02Z

Fix distributedlog-core findbug inconsistent synchronization warings




---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---


[GitHub] incubator-distributedlog pull request #51: (WIP) DL-81: Build the distribute...

2016-12-11 Thread xieliang
Github user xieliang commented on a diff in the pull request:


https://github.com/apache/incubator-distributedlog/pull/51#discussion_r91875966
  
--- Diff: website/community/release_guide.md ---
@@ -0,0 +1,507 @@
+---
+layout: default
+title: "DistributedLog Release Guide"
+permalink: /community/release-guide/
+---
+
+# Apache DistributedLog Release Guide
+
+* TOC
+{:toc}
+
+This page documents the procedure to make an Apache DistributedLog 
release. Creit to the 
[Beam](http://distributedlog.incubator.apache.org/contribute/release-guide/) 
project. We've borrow liberally from their documentation.
--- End diff --

seems replaced the beam link with DL's ... :)


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---