Github user kchilton2 commented on a diff in the pull request:
https://github.com/apache/incubator-rya/pull/172#discussion_r160788899
--- Diff:
extras/shell/src/main/java/org/apache/rya/shell/util/RyaDetailsFormatter.java
---
@@ -72,30 +72,28 @@ public String format(StorageType storageType, final
RyaDetails details) {
report.append(" Temporal Index:\n");
report.append(" Enabled: ").append(
details.getTemporalIndexDetails().isEnabled() ).append("\n");
- if(storageType == StorageType.ACCUMULO) {
- report.append(" PCJ Index:\n");
- final PCJIndexDetails pcjDetails =
details.getPCJIndexDetails();
- report.append(" Enabled: ").append( pcjDetails.isEnabled()
).append("\n");
- if(pcjDetails.isEnabled()) {
- if(pcjDetails.getFluoDetails().isPresent()) {
- final String fluoAppName =
pcjDetails.getFluoDetails().get().getUpdateAppName();
- report.append(" Fluo App Name:
").append(fluoAppName).append("\n");
- }
+ report.append(" PCJ Index:\n");
--- End diff --
The Statistics section does not have anything to do with PCJs so it
shouldn't be printed conditionally when PCJ is turned on. They are also an
Accumulo only feature.
---