asterixdb-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mb...@apache.org
Subject asterixdb git commit: Fail On Hard Tabs
Date Wed, 15 Jun 2016 15:11:59 GMT
Repository: asterixdb
Updated Branches:
  refs/heads/master 6c80d5333 -> d1e2a9953


Fail On Hard Tabs

Fail the build if hard tabs are encountered in source or resource
files.

Change-Id: I6af9e9fb72b2af51bc322e9912e7d1067a2a74c0
Reviewed-on: https://asterix-gerrit.ics.uci.edu/923
Reviewed-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>
Reviewed-by: abdullah alamoudi <bamousaa@gmail.com>


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

Branch: refs/heads/master
Commit: d1e2a9953420d262737dd2083ea73d8abcf7c955
Parents: 6c80d53
Author: Michael Blow <mblow@apache.org>
Authored: Tue Jun 14 12:27:46 2016 -0400
Committer: Michael Blow <michael.blow@couchbase.com>
Committed: Wed Jun 15 08:11:14 2016 -0700

----------------------------------------------------------------------
 .../src/test/resources/logging.properties       |  4 ++--
 .../logging.properties                          |  4 ++--
 .../src/test/resources/logging.properties       |  4 ++--
 asterixdb/pom.xml                               | 25 ++++++++++++++++++++
 hyracks-fullstack/pom.xml                       | 25 ++++++++++++++++++++
 5 files changed, 56 insertions(+), 6 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/asterixdb/blob/d1e2a995/asterixdb/asterix-app/src/test/resources/logging.properties
----------------------------------------------------------------------
diff --git a/asterixdb/asterix-app/src/test/resources/logging.properties b/asterixdb/asterix-app/src/test/resources/logging.properties
index 1253c5e..5b5e3b6 100644
--- a/asterixdb/asterix-app/src/test/resources/logging.properties
+++ b/asterixdb/asterix-app/src/test/resources/logging.properties
@@ -16,7 +16,7 @@
 # specific language governing permissions and limitations
 # under the License.
 ############################################################
-#  	Default Logging Configuration File
+#      Default Logging Configuration File
 #
 # You can use a different file by specifying a filename
 # with the java.util.logging.config.file system property.  
@@ -24,7 +24,7 @@
 ############################################################
 
 ############################################################
-#  	Global properties
+#      Global properties
 ############################################################
 
 # "handlers" specifies a comma separated list of log Handler 

http://git-wip-us.apache.org/repos/asf/asterixdb/blob/d1e2a995/asterixdb/asterix-experiments/src/main/resources/ingestion-experiment-binary-and-configs/logging.properties
----------------------------------------------------------------------
diff --git a/asterixdb/asterix-experiments/src/main/resources/ingestion-experiment-binary-and-configs/logging.properties
b/asterixdb/asterix-experiments/src/main/resources/ingestion-experiment-binary-and-configs/logging.properties
index e3c448c..c297232 100644
--- a/asterixdb/asterix-experiments/src/main/resources/ingestion-experiment-binary-and-configs/logging.properties
+++ b/asterixdb/asterix-experiments/src/main/resources/ingestion-experiment-binary-and-configs/logging.properties
@@ -13,7 +13,7 @@
 # limitations under the License.
 #*/
 ############################################################
-#  	Default Logging Configuration File
+#      Default Logging Configuration File
 #
 # You can use a different file by specifying a filename
 # with the java.util.logging.config.file system property.  
@@ -21,7 +21,7 @@
 ############################################################
 
 ############################################################
-#  	Global properties
+#      Global properties
 ############################################################
 
 # "handlers" specifies a comma separated list of log Handler 

http://git-wip-us.apache.org/repos/asf/asterixdb/blob/d1e2a995/asterixdb/asterix-tools/src/test/resources/logging.properties
----------------------------------------------------------------------
diff --git a/asterixdb/asterix-tools/src/test/resources/logging.properties b/asterixdb/asterix-tools/src/test/resources/logging.properties
index 33415ca..1d1bbda 100644
--- a/asterixdb/asterix-tools/src/test/resources/logging.properties
+++ b/asterixdb/asterix-tools/src/test/resources/logging.properties
@@ -16,7 +16,7 @@
 # specific language governing permissions and limitations
 # under the License.
 ############################################################
-#  	Default Logging Configuration File
+#      Default Logging Configuration File
 #
 # You can use a different file by specifying a filename
 # with the java.util.logging.config.file system property.  
@@ -24,7 +24,7 @@
 ############################################################
 
 ############################################################
-#  	Global properties
+#      Global properties
 ############################################################
 
 # "handlers" specifies a comma separated list of log Handler 

http://git-wip-us.apache.org/repos/asf/asterixdb/blob/d1e2a995/asterixdb/pom.xml
----------------------------------------------------------------------
diff --git a/asterixdb/pom.xml b/asterixdb/pom.xml
index ee0a277..7c5e3e7 100644
--- a/asterixdb/pom.xml
+++ b/asterixdb/pom.xml
@@ -216,6 +216,31 @@
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-resources-plugin</artifactId>
       </plugin>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-checkstyle-plugin</artifactId>
+        <version>2.17</version>
+        <executions>
+          <execution>
+            <id>verify-style</id>
+            <phase>process-classes</phase>
+            <goals>
+              <goal>check</goal>
+            </goals>
+          </execution>
+        </executions>
+        <configuration>
+          <logViolationsToConsole>true</logViolationsToConsole>
+          <checkstyleRules>
+            <module name="Checker">
+              <!-- Checks for whitespace                               -->
+              <!-- See http://checkstyle.sf.net/config_whitespace.html -->
+              <module name="FileTabCharacter"/>
+            </module>
+          </checkstyleRules>
+          <sourceDirectories>${project.build.sourceDirectory},${project.build.testSourceDirectory}</sourceDirectories>
+        </configuration>
+      </plugin>
     </plugins>
     <pluginManagement>
       <plugins>

http://git-wip-us.apache.org/repos/asf/asterixdb/blob/d1e2a995/hyracks-fullstack/pom.xml
----------------------------------------------------------------------
diff --git a/hyracks-fullstack/pom.xml b/hyracks-fullstack/pom.xml
index abf710e..8fd0502 100644
--- a/hyracks-fullstack/pom.xml
+++ b/hyracks-fullstack/pom.xml
@@ -281,6 +281,31 @@
           </execution>
         </executions>
       </plugin>
+      <plugin>
+        <groupId>org.apache.maven.plugins</groupId>
+        <artifactId>maven-checkstyle-plugin</artifactId>
+        <version>2.17</version>
+        <executions>
+          <execution>
+            <id>verify-style</id>
+            <phase>process-classes</phase>
+            <goals>
+              <goal>check</goal>
+            </goals>
+          </execution>
+        </executions>
+        <configuration>
+          <logViolationsToConsole>true</logViolationsToConsole>
+          <checkstyleRules>
+            <module name="Checker">
+              <!-- Checks for whitespace                               -->
+              <!-- See http://checkstyle.sf.net/config_whitespace.html -->
+              <module name="FileTabCharacter"/>
+            </module>
+          </checkstyleRules>
+          <sourceDirectories>${project.build.sourceDirectory},${project.build.testSourceDirectory}</sourceDirectories>
+        </configuration>
+      </plugin>
     </plugins>
     <pluginManagement>
       <plugins>


Mime
View raw message