Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #3287: Exclude "passwd" extensions from rat. #3288

Merged
merged 1 commit into from Jan 16, 2019

Conversation

tkaitchuck
Copy link
Member

Signed-off-by: Tom Kaitchuck tom.kaitchuck@emc.com

Change log description

  • Exclude passwd files from the build.

Purpose of the change
Fixes #3287

What the code does
Changes the rat configuration to exclude all files ending in 'passwd' as opposed to just those named 'passwd'

Signed-off-by: Tom Kaitchuck <tom.kaitchuck@emc.com>
@codecov
Copy link

codecov bot commented Jan 16, 2019

Codecov Report

Merging #3288 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #3288      +/-   ##
============================================
- Coverage     78.83%   78.81%   -0.03%     
- Complexity     8022     8024       +2     
============================================
  Files           609      609              
  Lines         31497    31497              
  Branches       3046     3046              
============================================
- Hits          24830    24823       -7     
- Misses         4868     4880      +12     
+ Partials       1799     1794       -5
Impacted Files Coverage Δ Complexity Δ
...server/attributes/ContainerAttributeIndexImpl.java 79.71% <0%> (-10.15%) 17% <0%> (-2%)
...ntstore/server/host/ZKSegmentContainerManager.java 87.5% <0%> (-6.25%) 6% <0%> (ø)
.../client/stream/impl/ControllerResolverFactory.java 82.56% <0%> (-2.76%) 7% <0%> (ø)
...avega/controller/server/ControllerServiceMain.java 80.45% <0%> (-1.15%) 14% <0%> (-1%)
...main/java/io/pravega/storage/hdfs/HDFSStorage.java 73.38% <0%> (-0.72%) 51% <0%> (ø)
.../segmentstore/server/writer/SegmentAggregator.java 78.5% <0%> (-0.14%) 236% <0%> (-2%)
...a/segmentstore/server/logs/OperationProcessor.java 85.13% <0%> (+0.37%) 39% <0%> (+1%) ⬆️
...o/pravega/segmentstore/server/logs/DurableLog.java 89.75% <0%> (+0.48%) 68% <0%> (+1%) ⬆️
...ent/netty/impl/ClientConnectionInboundHandler.java 53.19% <0%> (+1.06%) 13% <0%> (+1%) ⬆️
...tore/server/reading/RedirectedReadResultEntry.java 88.52% <0%> (+1.63%) 26% <0%> (+1%) ⬆️
... and 3 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4a02461...dc7221a. Read the comment docs.

@andreipaduroiu andreipaduroiu merged commit 9fe1985 into pravega:master Jan 16, 2019
@tkaitchuck tkaitchuck deleted the excludePasswd branch January 17, 2019 19:43
adrianmo added a commit to adrianmo/pravega that referenced this pull request Jan 23, 2019
* master: (31 commits)
  Issue 3279: Client is performing blocking operations inside Netty Event Loop (pravega#3280)
  Issue 3200: Document Review: wire-protocol.md (pravega#3201)
  Issue 2933: (SegmentStore) testEndToEndWithFencing fixes (pravega#3294)
  Issue pravega#2768: Rename PravegaCredsWrapper class and its member variable (pravega#3302)
  Issue pravega#3299: Reduce dependencies in client (pravega#3300)
  Issue 2895: Wire Protocol implementation for TableStore (pravega#3283)
  Issue-3239: Make DYNAMIC_LOGGER non-static (pravega#3293)
  Issue 2956: (SegmentStore) TableSegment.deleteIfEmpty (pravega#3263)
  Issue pravega#3289: Use compile-time dependencies where possible (pravega#3290)
  Issue 3223: RetentionTest failure (pravega#3272)
  Issue 2001: Make a defensive copy of streamCut's position map (pravega#3281)
  Issue 3287: Exclude "passwd" extensions from rat. (pravega#3288)
  Issue 3285: Changed image pull policy. (pravega#3286)
  Issue 3266: Ensure bookkeeper.bkAckQuorumSize is 3 for BookieFailover test. (pravega#3267)
  Issue 3264: Return a SegmentHandle from Storage create method. (pravega#3276)
  Issue 3086: Controller Diagrams(new) included (pravega#3097)
  Issue 2570: Update segment metrics when a transaction merge happens (pravega#3239)
  Issue 3149: (SegmentStore) Pinned Segments (pravega#3215)
  Issue 3269: Fix invalid initialization order in AbstractScaleTests. (pravega#3270)
  Issue 3148: (Segment Store) Segment Metadata Store (pravega#3206)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants