.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
gets
written/scyned unless dedup is implemented.
@sijie I am not sure what is the biggest concern about this. Are you worried
about too many configuration parameters? or something else?
This is an automated message from the Apach
only right?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.
better organized as
speculative reads has its own policy, but trying to understand what use-case
drove you to this change at Twitter?
This is an automated message from the Apache Git Service.
To respond to the message, please log
automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
place.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure a
place.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure a
implement this with
standard servlets.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service
he underneath problems and this flag will most likely be used anymore
when we have the correct solution in place.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use
>
--
Jvrao
---
First they ignore you, then they laugh at you, then they fight you, then
you win. - Mahatma Gandhi
This is an automated message from the Apache Git Service.
To respond to the message
aspects on the
solutions.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact
sijie closed issue #216: Remove useless and misleading '&= true'
URL: https://github.com/apache/bookkeeper/issues/216
This is an automated message from the Apache Git Service.
To respond to the message, please
sijie closed pull request #215: Issue 216: Remove useless and misleading '&=
true'
URL: https://github.com/apache/bookkeeper/pull/215
This is an automated message from the Apache Git Service.
To respond
sijie closed pull request #190: BOOKKEEPER-1034: Bookie start in RO when
diskfull
URL: https://github.com/apache/bookkeeper/pull/190
This is an automated message from the Apache Git Service.
To respond to the message
request.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at
sijie closed pull request #193: BOOKKEEPER-1086: ZkUnderreplicationManager
cache watcher
URL: https://github.com/apache/bookkeeper/pull/193
This is an automated message from the Apache Git Service.
To respond to the
the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git
watch = guard;
+ readAssertChildren("/z/a", new String[]{});
+} catch (Exception e) {
Review comment:
No, the exception is meant to fire and the test is that the watch is
canceled when it does (hence the assertNotFired below).
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git
GPG_KEY" \
+#&& gpg --batch --verify "$DISTRO_NAME.tar.gz.asc" "$DISTRO_NAME.tar.gz" \
+&& tar -xzf "$DISTRO_NAME.tar.gz" \
+&& rm -r "$GNUPGHOME" "$DISTRO_NAME.tar.gz" "$DISTRO_NAME.tar.gz.asc" \
message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
request`
Review comment:
@eolivelli does this address your comment? do we need to call out
'BOOKKEEPER-#' explicility?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitH
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards
leaking in current code base.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With
kay to you.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastruct
jiazhai commented on issue #205: Issue 208: Improve ledger fence logic
URL: https://github.com/apache/bookkeeper/pull/205#issuecomment-311847825
Seems there is conflict files.
This is an automated message from the Apache Git
Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
eolivelli closed pull request #213: Issue 212: change github PR template to
adjust both issue# and jira#
URL: https://github.com/apache/bookkeeper/pull/213
This is an automated message from the Apache Git Service.
To
eolivelli closed issue #212: Change pull request template to adjust both issue#
and jira#
URL: https://github.com/apache/bookkeeper/issues/212
This is an automated message from the Apache Git Service.
To respond to the
License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+---
+- scm:
+name: repo
+scm:
+- git:
+
eestyle project"
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With
findbugs:check`.
- [ ] Replace `` in the title with the actual
Issue/JIRA number.
---
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the
edger directory to exclude.
Review comment:
Add new @param fallback here.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
*NO* Checks to see the fullness of the randomly
picked dir. Any consequences ?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific
:
Comment is vague. Please reword it.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
add maven checkstyle plugin and attache it to the validate goal if you
would like.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific
other logFenceResult
here???
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please co
add maven checkstyle plugin and attache it to the validate goal if you
would like.
Or just make it runnable by ```mvn checkstyle:checkstyle```
This is an automated message from the Apache Git Service.
To respond to the message
new thread
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact
do.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us
ne 96..
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
it is necessary.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infras
it is and just going to change
the config variable name as we discussed earlier (from 'partition' to
'diskPartition') and resend the commit.
This is an automated message from the Apache Git Service.
To respond t
t out.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure a
have only one journal dir right?
@reddycharan what do you mean by " Will add check for journaldirectories " ?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and u
/123eccd435a4a96a9147ed4a24efbe9025fe79ba
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
concerned.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastruct
build-tools/src/main/resources/distributedlog/checkstyle.xml
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries
?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
. I have improved the comments.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please
edger directory to exclude.
Review comment:
done
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about
uests.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure a
releasing ByteBuf. I don't see that in master code.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries
(LongZkLedgerIdGenerator.java:320)
`
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact
close the connection, error
out the pending requests and cleaning up the resources.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specif
component, we can only release the buffer when the add
operation is completed.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific
, the trunk, of jenkins job builder,
the latest released version is somehow buggy and does handle the email
reporting plugin well
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
that @merlimat suggestion is good. Can you give it a try?
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries
:
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git
sijie closed issue #207: Change the HashSet to BitSet to handle duplicated
bookies on reading entries
URL: https://github.com/apache/bookkeeper/issues/207
This is an automated message from the Apache Git Service.
To
sijie closed pull request #199: Issue 207: change the HashSet to BitSet to
handle duplicated bookies on reading entries
URL: https://github.com/apache/bookkeeper/pull/199
This is an automated message from the Apache Git
sijie closed pull request #200: Issue 209: Introduce Speculative Read/Read LAC
policy
URL: https://github.com/apache/bookkeeper/pull/200
This is an automated message from the Apache Git Service.
To respond to the message
sijie closed issue #209: Introduce a Speculative Read/Read LAC policy
URL: https://github.com/apache/bookkeeper/issues/209
This is an automated message from the Apache Git Service.
To respond to the message, please log on
sijie closed pull request #202: BOOKKEEPER-759: Delay Ensemble Change & Disable
Ensemble Change
URL: https://github.com/apache/bookkeeper/pull/202
This is an automated message from the Apache Git Service.
To respond to
also have a speculative mechanism for long poll reads.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about
from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
case #221
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
'1' for this failure.
@eolivelli
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
'1' for this failure.
@eolivelli
----
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
otent.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure a
observer) throws IOException;
Review comment:
same typo previoisLAC
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment
, long
previoisLAC, Observer observer) throws IOException {
Review comment:
minor typo: previoisLAC
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
eolivelli closed pull request #214: Fix ByteBuf refcount release
URL: https://github.com/apache/bookkeeper/pull/214
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub
)
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
sijie commented on issue #222: Issue 221: Make netty server shutdown idempotent
URL: https://github.com/apache/bookkeeper/pull/222#issuecomment-312200766
/cc @reddycharan
This is an automated message from the Apache Git
s with the switch between
normal reads and long poll reads when detecting lac is advancing. once this
change is in, distributedlog should be compatible with current community master.
This is an automated message from the Apache G
e will not
print twice "Shutting down" and doing to isRunning.set
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the spe
this topic
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastruct
the change
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.
having it merged in BK ! Thank you
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
x" in the "[ ]".
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this ser
-alpine
+MAINTAINER Francesco Caliumi
+
+# Install required packages
+RUN apk add --no-cache \
+bash \
+su-exec
+
+ENV ZK_SERVERS= \
Review comment:
+1. It is a great py.
This is an automated message from the Apache Git
methods should be idempotent too but if it
is possible it is better to make the call to "close" once. It is cleaner and
for the long term it is the best approch
This is an automated message from the Apache Git Service.
To
.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With
started working on the CheckStyle, but there are lots of errors, I started
to fix, so it is in progress. I'll create an issue and a different PR for that
at the weekend.
This is an automated message from the Apache Git Servic
be called
twice, I would like to have logging to tell, so if we want to debug, we know
what happened.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to
d twice, I
would like it to be set eventually.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this servi
eolivelli commented on issue #222: Issue 221: Make netty server shutdown
idempotent
URL: https://github.com/apache/bookkeeper/pull/222#issuecomment-312301512
Ok for me
This is an automated message from the Apache Git
nt to provide the reopenable log. And
thats one of the reasons that we think we want to merge the library back to
provide an easier library for access.
This is an automated message from the Apache Git Service.
To respond to the me
using on merging
big feature changes to make sure 4.5 move forward.
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
just collected all
the pieces and I am waiting to have the native BK library.
I can wait
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the
1 - 100 of 1415 matches
Mail list logo
