[
https://issues.apache.org/jira/browse/YETUS-124?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kengo Seki updated YETUS-124:
-----------------------------
Attachment: YETUS-124.00.patch
Attaching a patch. I tested it for some issues and it worked well.
{code}
[sekikn@mobile yetus]$ MAVEN_OPTS=-XX:MaxPermSize=512m precommit/test-patch.sh
--basedir=../drill --project=drill --resetrepo DRILL-3522
/tmp/yetus-28505.30005 has been created
Running in destructive (--resetrepo) developer mode
Processing: DRILL-3522
DRILL-3522 patch is being downloaded at Fri Oct 30 23:31:57 JST 2015 from
https://issues.apache.org/jira/secure/attachment/12746251/DRILL-3522.1.patch.txt
(snip)
| Vote | Subsystem | Runtime | Comment
============================================================================
| +1 | @author | 0m 00s | The patch does not contain any @author
| | | | tags.
| -1 | test4tests | 0m 00s | The patch doesn't appear to include any
| | | | new or modified tests. Please justify why
| | | | no new tests are needed for this patch.
| | | | Also please list what manual steps were
| | | | performed to verify this patch.
| +1 | mvninstall | 5m 39s | master passed
| +1 | compile | 0m 29s | master passed
| +1 | checkstyle | 0m 09s | master passed
| +1 | mvneclipse | 0m 25s | master passed
| +1 | javadoc | 0m 18s | master passed
| +1 | mvninstall | 0m 33s | the patch passed
| +1 | compile | 0m 31s | the patch passed
| +1 | javac | 0m 31s | the patch passed
| -1 | checkstyle | 0m 10s | Patch generated 1 new checkstyle issues
| | | | in contrib/storage-mongo (total was 37,
| | | | now 37).
| +1 | mvneclipse | 0m 23s | the patch passed
| +1 | whitespace | 0m 00s | Patch has no whitespace issues.
| +1 | findbugs | 0m 56s | the patch passed
| +1 | javadoc | 0m 17s | the patch passed
| +1 | asflicense | 0m 08s | Patch does not generate ASF License
| | | | warnings.
| | | 11m 10s |
|| Subsystem || Report/Notes ||
============================================================================
| JIRA Patch URL |
https://issues.apache.org/jira/secure/attachment/12746251/DRILL-3522.1.patch.txt
|
| JIRA Issue | DRILL-3522 |
| Optional Tests | asflicense javac javadoc unit findbugs checkstyle
compile |
| uname | Darwin mobile.local 15.0.0 Darwin Kernel Version 15.0.0: Sat Sep 19
15:53:46 PDT 2015; root:xnu-3247.10.11~1/RELEASE_X86_64 x86_64 |
| Build tool | maven |
| Personality | /Users/sekikn/yetus/precommit/personality/drill.sh |
| git revision | master / e4b94a7 |
| Default Java | 1.7.0_80 |
| findbugs | v3.0.1 |
| checkstyle |
/private/tmp/yetus-28505.30005/diff-checkstyle-contrib_storage-mongo.txt |
| Max memory used | 129MB |
| Powered by | Apache Yetus http://yetus.apache.org |
{code}
{code}
[sekikn@mobile yetus]$ MAVEN_OPTS=-XX:MaxPermSize=512m precommit/test-patch.sh
--basedir=../drill --project=drill --resetrepo DRILL-3871
/tmp/yetus-25178.28676 has been created
Running in destructive (--resetrepo) developer mode
Processing: DRILL-3871
DRILL-3871 appears to be a Github PR. Switching Modes.
GITHUB PR #219 is being downloaded at Fri Oct 30 23:43:06 JST 2015 from
https://github.com/apache/drill/pull/219
Patch from GITHUB PR #219 is being downloaded at Fri Oct 30 23:43:08 JST 2015
from
https://github.com/apache/drill/pull/219.patch
(snip)
| Vote | Subsystem | Runtime | Comment
============================================================================
| +1 | @author | 0m 00s | The patch does not contain any @author
| | | | tags.
| +1 | test4tests | 0m 00s | The patch appears to include 7 new or
| | | | modified test files.
| +1 | mvninstall | 5m 23s | master passed
| +1 | compile | 2m 06s | master passed
| +1 | checkstyle | 0m 23s | master passed
| +1 | mvneclipse | 0m 35s | master passed
| +1 | javadoc | 8m 18s | master passed
| +1 | mvninstall | 2m 41s | the patch passed
| +1 | compile | 1m 59s | the patch passed
| +1 | javac | 1m 59s | the patch passed
| -1 | checkstyle | 0m 21s | Patch generated 9 new checkstyle issues
| | | | in exec/java-exec (total was 66, now 33).
| +1 | mvneclipse | 0m 28s | the patch passed
| +1 | whitespace | 0m 00s | Patch has no whitespace issues.
| +1 | findbugs | 4m 10s | the patch passed
| +1 | javadoc | 9m 05s | the patch passed
| +1 | asflicense | 0m 15s | Patch does not generate ASF License
| | | | warnings.
| | | 40m 10s |
|| Subsystem || Report/Notes ||
============================================================================
| JIRA Issue | DRILL-3871 |
| GITHUB PR | https://github.com/apache/drill/pull/219 |
| Optional Tests | asflicense javac javadoc unit findbugs checkstyle
compile |
| uname | Darwin mobile.local 15.0.0 Darwin Kernel Version 15.0.0: Sat Sep 19
15:53:46 PDT 2015; root:xnu-3247.10.11~1/RELEASE_X86_64 x86_64 |
| Build tool | maven |
| Personality | /Users/sekikn/yetus/precommit/personality/drill.sh |
| git revision | master / e4b94a7 |
| Default Java | 1.7.0_80 |
| findbugs | v3.0.1 |
| checkstyle |
/private/tmp/yetus-25178.28676/diff-checkstyle-exec_java-exec.txt |
| Max memory used | 128MB |
| Powered by | Apache Yetus http://yetus.apache.org |
{code}
> add exemplar drill personality
> ------------------------------
>
> Key: YETUS-124
> URL: https://issues.apache.org/jira/browse/YETUS-124
> Project: Yetus
> Issue Type: Task
> Components: Test Patch
> Affects Versions: 0.1.0
> Reporter: Sean Busbey
> Attachments: YETUS-124.00.patch
>
>
> Drill is looking to improve their precommit testing. main concern is speed,
> so maybe we can help depending on where the slow down is.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
