You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/09/22 19:02:55 UTC
[GitHub] [accumulo-testing] jmark99 opened a new pull request #124: Missing license headers #122
jmark99 opened a new pull request #124:
URL: https://github.com/apache/accumulo-testing/pull/124
* Added missing license headers to 20+ files.
* Added the apache-rat plugin to the pom file.
* Moved the splits data files in test/bench/lib into a data
subdirectory.
* Modified TableSplitsBenchmark.py to account for the movement of the
splits file into the data directory.
* Added exclusions for the data directory and the exclude_classes.txt
file.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] ctubbsii commented on a change in pull request #124: Missing license headers #122
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on a change in pull request #124:
URL: https://github.com/apache/accumulo-testing/pull/124#discussion_r492992338
##########
File path: .gitignore
##########
@@ -5,3 +5,5 @@
/*.iml
/.idea
/logs/
+/test/bench/lib/*.pyc
+/test/bench/cloudstone*/*.pyc
Review comment:
This might be better as:
```suggestion
*.pyc
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] ctubbsii commented on a change in pull request #124: Missing license headers #122
Posted by GitBox <gi...@apache.org>.
ctubbsii commented on a change in pull request #124:
URL: https://github.com/apache/accumulo-testing/pull/124#discussion_r492992338
##########
File path: .gitignore
##########
@@ -5,3 +5,5 @@
/*.iml
/.idea
/logs/
+/test/bench/lib/*.pyc
+/test/bench/cloudstone*/*.pyc
Review comment:
This might be better as:
```suggestion
*.pyc
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] jmark99 merged pull request #124: Missing license headers #122
Posted by GitBox <gi...@apache.org>.
jmark99 merged pull request #124:
URL: https://github.com/apache/accumulo-testing/pull/124
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo-testing] jmark99 merged pull request #124: Missing license headers #122
Posted by GitBox <gi...@apache.org>.
jmark99 merged pull request #124:
URL: https://github.com/apache/accumulo-testing/pull/124
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org