[ 
https://issues.apache.org/jira/browse/DRILL-4604?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15258501#comment-15258501
 ] 

ASF GitHub Bot commented on DRILL-4604:
---------------------------------------

Github user sudheeshkatkam commented on a diff in the pull request:

    https://github.com/apache/drill/pull/482#discussion_r61127476
  
    --- Diff: 
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/DrillRoot.java 
---
    @@ -55,44 +59,89 @@ public Viewable getStats() {
       @GET
       @Path("/stats.json")
       @Produces(MediaType.APPLICATION_JSON)
    -  public List<Stat> getStatsJSON() {
    -    List<Stat> stats = Lists.newLinkedList();
    -    stats.add(new Stat("Number of Drill Bits", 
work.getContext().getBits().size()));
    -    int number = 0;
    -    for (CoordinationProtos.DrillbitEndpoint bit : 
work.getContext().getBits()) {
    -      String initialized = bit.isInitialized() ? " initialized" : " not 
initialized";
    -      stats.add(new Stat("Bit #" + number, bit.getAddress() + 
initialized));
    -      ++number;
    +  public Stats getStatsJSON() {
    +    String version = 
work.getContext().getOptionManager().getOption(ExecConstants.CLUSTER_VERSION).string_val;
    --- End diff --
    
    The assumption for system options is that default values are same across 
drillbits, and so they need not be put in the persistent store.
    
    In this case, the default values across drillbits might not be the same. 
This option has a default value of what is returned by 
`DrillVersionInfo.getVersion()`, and default values are **not put** in 
persistent store. But if an admin modifies the option, the value is put in the 
store.


> Generate warning on Web UI if drillbits version mismatch is detected
> --------------------------------------------------------------------
>
>                 Key: DRILL-4604
>                 URL: https://issues.apache.org/jira/browse/DRILL-4604
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.6.0
>            Reporter: Arina Ielchiieva
>            Assignee: Arina Ielchiieva
>             Fix For: 1.7.0
>
>         Attachments: index_page.JPG, index_page_mismatch.JPG
>
>
> Display drillbit version on web UI. If any of drillbits version doesn't match 
> with current drillbit, generate warning.
> Screenshots - TBA.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to