You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by da...@apache.org on 2017/08/26 01:43:54 UTC

[1/3] kudu git commit: KUDU-1544: Race in Java client's AsyncKuduSession.apply()

Repository: kudu
Updated Branches:
  refs/heads/master faa0b14ef -> 1c70e5df8


KUDU-1544: Race in Java client's AsyncKuduSession.apply()

This fixes a potential race in AsyncKuduSession.apply by acquiring the
notification deferred early. See the JIRA for details about the race.
Acquiring the notification deferred early is always safe, the only
downside is that clients may be spuriously notified that there is buffer
space available when in fact there is not.

Change-Id: I1791f5ed51e2d79cd9e9bcbf83e9b30eede871b4
Reviewed-on: http://gerrit.cloudera.org:8080/7839
Reviewed-by: Adar Dembo <ad...@cloudera.com>
Tested-by: Kudu Jenkins


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/3cd95f19
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/3cd95f19
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/3cd95f19

Branch: refs/heads/master
Commit: 3cd95f190f286b64b9b68f0fd3c925fd79bdcae9
Parents: faa0b14
Author: Dan Burkert <da...@apache.org>
Authored: Fri Aug 25 15:54:03 2017 -0700
Committer: Dan Burkert <da...@apache.org>
Committed: Sat Aug 26 01:35:09 2017 +0000

----------------------------------------------------------------------
 .../main/java/org/apache/kudu/client/AsyncKuduSession.java    | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/3cd95f19/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
----------------------------------------------------------------------
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
index 551faf6..b23cbba 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
@@ -543,6 +543,7 @@ public class AsyncKuduSession implements SessionConfiguration {
     Buffer fullBuffer = null;
     try {
       synchronized (monitor) {
+        Deferred<Void> notification = flushNotification.get();
         if (activeBuffer == null) {
           // If the active buffer is null then we recently flushed. Check if there
           // is an inactive buffer available to replace as the active.
@@ -554,7 +555,7 @@ public class AsyncKuduSession implements SessionConfiguration {
             // This can happen if the user writes into a buffer, flushes it, writes
             // into the second, flushes it, and immediately tries to write again.
             throw new PleaseThrottleException(statusServiceUnavailable,
-                                              null, operation, flushNotification.get());
+                                              null, operation, notification);
           }
         }
 
@@ -582,7 +583,7 @@ public class AsyncKuduSession implements SessionConfiguration {
               Status statusServiceUnavailable =
                   Status.ServiceUnavailable("All buffers are currently flushing");
               throw new PleaseThrottleException(statusServiceUnavailable,
-                                                null, operation, flushNotification.get());
+                                                null, operation, notification);
             }
           }
 
@@ -600,7 +601,7 @@ public class AsyncKuduSession implements SessionConfiguration {
                   Status.ServiceUnavailable("The previous buffer hasn't been flushed and the " +
                       "current buffer is over the low watermark, please retry later");
               throw new PleaseThrottleException(statusServiceUnavailable,
-                                                null, operation, flushNotification.get());
+                                                null, operation, notification);
             }
           }
 


[3/3] kudu git commit: release notes: add contributor counts

Posted by da...@apache.org.
release notes: add contributor counts

Change-Id: Ib385b93671056c37474367266291370d7893550c
Reviewed-on: http://gerrit.cloudera.org:8080/7848
Reviewed-by: Todd Lipcon <to...@apache.org>
Tested-by: Todd Lipcon <to...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/1c70e5df
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/1c70e5df
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/1c70e5df

Branch: refs/heads/master
Commit: 1c70e5df829c2fbb4db6307627f8e096a6da8e08
Parents: 017fdcf
Author: Dan Burkert <da...@apache.org>
Authored: Fri Aug 25 17:03:19 2017 -0700
Committer: Dan Burkert <da...@apache.org>
Committed: Sat Aug 26 01:35:17 2017 +0000

----------------------------------------------------------------------
 docs/release_notes.adoc | 14 ++++++++++++++
 1 file changed, 14 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/1c70e5df/docs/release_notes.adoc
----------------------------------------------------------------------
diff --git a/docs/release_notes.adoc b/docs/release_notes.adoc
index c02af5b..0754455 100644
--- a/docs/release_notes.adoc
+++ b/docs/release_notes.adoc
@@ -119,6 +119,20 @@ on wire compatibility between Kudu 1.5 and versions earlier than 1.3:
 Please refer to the link:known_issues.html[Known Issues and Limitations] section of the
 documentation.
 
+[[rn_1.5.0_contributors]]
+
+Kudu 1.5 includes contributions from twenty-four people, including six
+first-time contributors:
+
+  - Abhishek Talluri
+  - Edward Fancher
+  - Michael Ho
+  - Sam Okrent
+  - Sandish Kumar
+  - Sri Sai Kumar Ravipati
+
+Thanks!
+
 [[resources_and_next_steps]]
 
 == Resources


[2/3] kudu git commit: release notes: add web UI config dashboard entry

Posted by da...@apache.org.
release notes: add web UI config dashboard entry

Change-Id: I8812e1643afc50af0633343aa76fbea79bc83d72
Reviewed-on: http://gerrit.cloudera.org:8080/7847
Tested-by: Kudu Jenkins
Reviewed-by: Todd Lipcon <to...@apache.org>


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/017fdcfb
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/017fdcfb
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/017fdcfb

Branch: refs/heads/master
Commit: 017fdcfb2bf10a666d88edef56c2894be77830ea
Parents: 3cd95f1
Author: Dan Burkert <da...@apache.org>
Authored: Fri Aug 25 16:56:48 2017 -0700
Committer: Dan Burkert <da...@apache.org>
Committed: Sat Aug 26 01:35:14 2017 +0000

----------------------------------------------------------------------
 docs/release_notes.adoc | 5 +++++
 1 file changed, 5 insertions(+)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/017fdcfb/docs/release_notes.adoc
----------------------------------------------------------------------
diff --git a/docs/release_notes.adoc b/docs/release_notes.adoc
index fda1a76..c02af5b 100644
--- a/docs/release_notes.adoc
+++ b/docs/release_notes.adoc
@@ -63,6 +63,11 @@
 [[rn_1.5.0_new_features]]
 == New features
 
+* Kudu server web UIs have a new configuration dashboard (/config) which gives a
+  a high level view of important security configuration values. The dashboard
+  gives administrators a quick way to validate security settings. More types of
+  configuration will be added in the future.
+
 == Optimizations and improvements
 
 [[rn_1.5.0_fixed_issues]]