You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2019/07/12 18:59:13 UTC
[kudu] branch branch-1.10.x updated (567782e -> 2350d68)
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a change to branch branch-1.10.x
in repository https://gitbox.apache.org/repos/asf/kudu.git.
from 567782e [docs] Update the site build to use the release directory
new f72998d Autogenerate the release index.md file
new bc0aad2 Remove experimental tags from NVM block cache gflags
new eb4e1f8 [docs] Fix quickstart IP one liner
new 2350d68 [doxygen] fix KuduScanTokenBuilder::SetDiffScan (take two)
The 4 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails. The revisions
listed as "add" were already present in the repository and have only
been added to this reference.
Summary of changes:
docs/quickstart.adoc | 2 +-
docs/support/scripts/index_template.md | 56 ++++++++++++++++++++++++++++++++++
docs/support/scripts/make_site.sh | 10 ++----
src/kudu/cfile/block_cache.cc | 3 +-
src/kudu/client/client.h | 17 +----------
src/kudu/util/nvm_cache.cc | 1 -
6 files changed, 62 insertions(+), 27 deletions(-)
create mode 100644 docs/support/scripts/index_template.md
[kudu] 01/04: Autogenerate the release index.md file
Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a commit to branch branch-1.10.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit f72998d476e6a6143fe2ad5147d2caadc6958635
Author: Grant Henke <gr...@apache.org>
AuthorDate: Thu Jul 11 08:16:00 2019 -0500
Autogenerate the release index.md file
When building the docs for a release, this patch
also builds the index.md file.
Change-Id: If93fe00aa0dd855d8f8fafa22c786747d2ae50ed
Reviewed-on: http://gerrit.cloudera.org:8080/13844
Reviewed-by: Alexey Serbin <as...@cloudera.com>
Tested-by: Kudu Jenkins
(cherry picked from commit 99d1bc82034698b5fc7d74e4bd2117a2ce9fc96b)
Reviewed-on: http://gerrit.cloudera.org:8080/13847
Tested-by: Grant Henke <gr...@apache.org>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
---
docs/support/scripts/index_template.md | 56 ++++++++++++++++++++++++++++++++++
docs/support/scripts/make_site.sh | 10 ++----
2 files changed, 59 insertions(+), 7 deletions(-)
diff --git a/docs/support/scripts/index_template.md b/docs/support/scripts/index_template.md
new file mode 100644
index 0000000..c1886b3
--- /dev/null
+++ b/docs/support/scripts/index_template.md
@@ -0,0 +1,56 @@
+---
+title: Apache Kudu release @@KUDU_VERSION@@
+layout: single_col
+active_nav: download
+single_col_extra_classes: releases
+---
+
+<!--
+
+Licensed to the Apache Software Foundation (ASF) under one
+or more contributor license agreements. See the NOTICE file
+distributed with this work for additional information
+regarding copyright ownership. The ASF licenses this file
+to you under the Apache License, Version 2.0 (the
+"License"); you may not use this file except in compliance
+with the License. You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing,
+software distributed under 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.
+
+-->
+
+## Apache Kudu release @@KUDU_VERSION@@
+
+See the [Kudu @@KUDU_VERSION@@ Release Notes](docs/release_notes.html).
+
+Downloads of Kudu @@KUDU_VERSION@@ are available in the following formats:
+
+* [Kudu @@KUDU_VERSION@@ source tarball](http://www.apache.org/dyn/closer.cgi?path=kudu/@@KUDU_VERSION@@/apache-kudu-@@KUDU_VERSION@@.tar.gz)
+ ([SHA512](https://www.apache.org/dist/kudu/@@KUDU_VERSION@@/apache-kudu-@@KUDU_VERSION@@.tar.gz.sha512),
+ [Signature](https://www.apache.org/dist/kudu/@@KUDU_VERSION@@/apache-kudu-@@KUDU_VERSION@@.tar.gz.asc))
+
+You can use the [KEYS file](https://www.apache.org/dist/kudu/KEYS) to verify the included GPG signature.
+
+To verify the integrity of the release, check the following:
+
+* Verify the checksum by downloading the release and the `.sha512` file, and
+ running the following command:
+ * On Linux: `sha512sum -c apache-kudu-@@KUDU_VERSION@@.tar.gz.sha512`
+ * On MacOS: `shasum -a 512 -c apache-kudu-@@KUDU_VERSION@@.tar.gz.sha512`
+* Verify the signature by downloading the release and the `.asc` file, and
+ doing the following:
+ * Import the KEYS file to the GPG keychain by running `gpg --import KEYS`
+ * Run `gpg --verify apache-kudu-@@KUDU_VERSION@@.tar.gz.asc apache-kudu-@@KUDU_VERSION@@.tar.gz`
+
+Additional links:
+
+* [Kudu @@KUDU_VERSION@@ Documentation](docs/)
+* [Kudu @@KUDU_VERSION@@ Java API docs](apidocs/)
+* [Kudu @@KUDU_VERSION@@ C++ API docs](cpp-client-api/)
\ No newline at end of file
diff --git a/docs/support/scripts/make_site.sh b/docs/support/scripts/make_site.sh
index 0d7d738..1a76e05 100755
--- a/docs/support/scripts/make_site.sh
+++ b/docs/support/scripts/make_site.sh
@@ -161,13 +161,9 @@ if [ -n "$OPT_DOXYGEN" ]; then
fi
fi
-SITE_SUBDIRS="docs"
-if [ -n "$OPT_JAVADOC" ]; then
- SITE_SUBDIRS="$SITE_SUBDIRS $JAVADOC_SUBDIR"
-fi
-if [ -n "$OPT_DOXYGEN" ]; then
- SITE_SUBDIRS="$SITE_SUBDIRS $CPP_CLIENT_API_SUBDIR"
-fi
+# Generate the release index file.
+sed "s/@@KUDU_VERSION@@/$VERSION/g" "$SOURCE_ROOT/docs/support/scripts/index_template.md" > \
+ "$RELEASE_OUTPUT_DIR/index.md"
cd "$SITE_OUTPUT_DIR"
SITE_ARCHIVE="$SITE_OUTPUT_DIR/website_archive.zip"
[kudu] 04/04: [doxygen] fix KuduScanTokenBuilder::SetDiffScan (take
two)
Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a commit to branch branch-1.10.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit 2350d68e4b85bc6dc32bbbe6d610f1f8f6fa1172
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Wed Jul 10 16:49:19 2019 -0700
[doxygen] fix KuduScanTokenBuilder::SetDiffScan (take two)
Since both {KuduScanner,KuduScanTokenBuilder}::SetDiffScan() are both
under the same PRIVATE_API conditional, it's possible to use the
@copydoc to avoid the duplication of the description.
This is a follow-up to 0b4948ea3.
Change-Id: I2de45bf62bd8ef240b0d0e0cd3f887ef3d40cd04
Reviewed-on: http://gerrit.cloudera.org:8080/13841
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <gr...@apache.org>
(cherry picked from commit 26bd0fdc4ba74d130f709837e33eb258633c51ce)
Reviewed-on: http://gerrit.cloudera.org:8080/13843
Tested-by: Grant Henke <gr...@apache.org>
Reviewed-by: Alexey Serbin <as...@cloudera.com>
---
src/kudu/client/client.h | 17 +----------------
1 file changed, 1 insertion(+), 16 deletions(-)
diff --git a/src/kudu/client/client.h b/src/kudu/client/client.h
index 20c3440..fdc408d 100644
--- a/src/kudu/client/client.h
+++ b/src/kudu/client/client.h
@@ -2432,24 +2432,9 @@ class KUDU_EXPORT KuduScanTokenBuilder {
Status SetSnapshotRaw(uint64_t snapshot_timestamp) WARN_UNUSED_RESULT;
/// @cond PRIVATE_API
-
- /// Set the start and end timestamp for a diff scan. The timestamps should be
- /// encoded HT timestamps.
- ///
- /// Additionally sets any other scan properties required by diff scans.
- ///
- /// Private API.
- ///
- /// @param [in] start_timestamp
- /// Start timestamp to set in raw encoded form
- /// (i.e. as returned by a previous call to a server).
- /// @param [in] end_timestamp
- /// End timestamp to set in raw encoded form
- /// (i.e. as returned by a previous call to a server).
- /// @return Operation result status.
+ /// @copydoc KuduScanner::SetDiffScan
Status SetDiffScan(uint64_t start_timestamp, uint64_t end_timestamp)
WARN_UNUSED_RESULT KUDU_NO_EXPORT;
-
/// @endcond
/// @copydoc KuduScanner::SetTimeoutMillis
[kudu] 03/04: [docs] Fix quickstart IP one liner
Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a commit to branch branch-1.10.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit eb4e1f8de8df5bf1aecb3b2767fe87bfdb8534ea
Author: Grant Henke <gr...@apache.org>
AuthorDate: Thu Jul 11 16:16:29 2019 -0500
[docs] Fix quickstart IP one liner
It was reported that on Linux an extra whitespace could
be included breaking the one liner. This uses `awk` instead
so exact spaces don't matter.
Change-Id: Ie401578da12157b802752ff62885c4b4dc6b3a29
Reviewed-on: http://gerrit.cloudera.org:8080/13848
Tested-by: Grant Henke <gr...@apache.org>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
(cherry picked from commit c1d16856bd4b3bb4b6242fa098d600b6ef321e6a)
Reviewed-on: http://gerrit.cloudera.org:8080/13853
---
docs/quickstart.adoc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/docs/quickstart.adoc b/docs/quickstart.adoc
index 9ccaafd..3f775a8 100644
--- a/docs/quickstart.adoc
+++ b/docs/quickstart.adoc
@@ -60,7 +60,7 @@ Set the `KUDU_QUICKSTART_IP` environment variable to your ip address:
[source,bash]
----
-$ export KUDU_QUICKSTART_IP=$(ifconfig | grep "inet " | grep -Fv 127.0.0.1 | cut -d" " -f2 | tail -1)
+$ export KUDU_QUICKSTART_IP=$(ifconfig | grep "inet " | grep -Fv 127.0.0.1 | awk '{print $2}' | tail -1)
----
=== Bring up the Cluster
[kudu] 02/04: Remove experimental tags from NVM block cache gflags
Posted by gr...@apache.org.
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a commit to branch branch-1.10.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit bc0aad22688e325287b644bff94e17d3bec70d05
Author: Adar Dembo <ad...@cloudera.com>
AuthorDate: Tue Jul 9 15:47:37 2019 -0700
Remove experimental tags from NVM block cache gflags
The block cache has supported non-volatile memory for almost four years
now, though one needed to configure some experimental gflags in order to
use it. The implementation was recently changed to use the newer memkind
library and, in the process, underwent additional testing on NVM hardware.
As such, I think it's time to remove the experimental tags to communicate to
users that the underlying code is supported upstream.
Note: I'm not marking them as 'stable' so that we retain the flexibility to
modify the configuration surface itself, if we find we need that.
Change-Id: Ie6b58118116ecc11b2eaa5cec4a9e72c29ac27f8
Reviewed-on: http://gerrit.cloudera.org:8080/13830
Reviewed-by: Greg Solovyev <gs...@cloudera.com>
Reviewed-by: Andrew Wong <aw...@cloudera.com>
Tested-by: Kudu Jenkins
(cherry picked from commit daf2572ad107bc50bb8a4e534a5e1414aa71bbe6)
Reviewed-on: http://gerrit.cloudera.org:8080/13838
---
src/kudu/cfile/block_cache.cc | 3 +--
src/kudu/util/nvm_cache.cc | 1 -
2 files changed, 1 insertion(+), 3 deletions(-)
diff --git a/src/kudu/cfile/block_cache.cc b/src/kudu/cfile/block_cache.cc
index ff17524..5ea8316 100644
--- a/src/kudu/cfile/block_cache.cc
+++ b/src/kudu/cfile/block_cache.cc
@@ -54,8 +54,7 @@ DEFINE_string(block_cache_type, "DRAM",
"Which type of block cache to use for caching data. "
"Valid choices are 'DRAM' or 'NVM'. DRAM, the default, "
"caches data in regular memory. 'NVM' caches data "
- "in a memory-mapped file using the NVML library.");
-TAG_FLAG(block_cache_type, experimental);
+ "in a memory-mapped file using the memkind library.");
using strings::Substitute;
diff --git a/src/kudu/util/nvm_cache.cc b/src/kudu/util/nvm_cache.cc
index 55adffd..320048b 100644
--- a/src/kudu/util/nvm_cache.cc
+++ b/src/kudu/util/nvm_cache.cc
@@ -59,7 +59,6 @@ DECLARE_bool(cache_force_single_shard);
DEFINE_string(nvm_cache_path, "/pmem",
"The path at which the NVM cache will try to allocate its memory. "
"This can be a tmpfs or ramfs for testing purposes.");
-TAG_FLAG(nvm_cache_path, experimental);
DEFINE_int32(nvm_cache_allocation_retry_count, 10,
"The number of times that the NVM cache will retry attempts to allocate "