[arrow-ballista] branch master updated: MINOR: Fix yarn warnings (#415)

[agrove]

This is an automated email from the ASF dual-hosted git repository.

agrove pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow-ballista.git


The following commit(s) were added to refs/heads/master by this push:
     new f899f101 MINOR: Fix yarn warnings (#415)
f899f101 is described below

commit f899f101ddd4655ec168cce89653eb31651c3065
Author: Andy Grove <andygrov...@gmail.com>
AuthorDate: Fri Oct 21 07:51:37 2022 -0600

    MINOR: Fix yarn warnings (#415)
    
    * fix yarn warnings
    
    * fix yarn warnings
    
    * fix another warning
---
 .github/workflows/typescript.yml                     | 2 +-
 ballista/ui/scheduler/src/components/Header.tsx      | 2 +-
 ballista/ui/scheduler/src/components/QueriesList.tsx | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/typescript.yml b/.github/workflows/typescript.yml
index e5e4ec59..a1b95821 100644
--- a/.github/workflows/typescript.yml
+++ b/.github/workflows/typescript.yml
@@ -38,4 +38,4 @@ jobs:
         run: |
           cd ballista/ui/scheduler
           yarn install
-          CI=false yarn build # TODO: fix warnings then remove CI=false
+          yarn build
\ No newline at end of file
diff --git a/ballista/ui/scheduler/src/components/Header.tsx 
b/ballista/ui/scheduler/src/components/Header.tsx
index d0dfaab0..dc0bb4ad 100644
--- a/ballista/ui/scheduler/src/components/Header.tsx
+++ b/ballista/ui/scheduler/src/components/Header.tsx
@@ -16,7 +16,7 @@
 // under the License.
 
 import React from "react";
-import { Box, Flex, Text, Button } from "@chakra-ui/react";
+import { Box, Flex, Button } from "@chakra-ui/react";
 import Logo from "./logo.svg";
 import { AiFillGithub, HiDocumentText } from "react-icons/all";
 import { SchedulerState } from "./Summary";
diff --git a/ballista/ui/scheduler/src/components/QueriesList.tsx 
b/ballista/ui/scheduler/src/components/QueriesList.tsx
index 582dc4a6..5fbcc8af 100644
--- a/ballista/ui/scheduler/src/components/QueriesList.tsx
+++ b/ballista/ui/scheduler/src/components/QueriesList.tsx
@@ -80,7 +80,7 @@ export const ActionsCell: (props: any) => React.ReactNode = 
(props: any) => {
     if (isOpen) {
       dot_svg("/api/job/" + props.value.job_id + "/dot_svg");
     }
-  }, [ref.current, dot_data, isOpen]);
+  }, [dot_data, isOpen, props.value.job_id]);
 
   const handleDownload = (url: string, filename: string) => {
     fetch(url, {
@@ -164,7 +164,7 @@ export const JobLinkCell: (props: any) => React.ReactNode = 
(props: any) => {
       getStages("/api/job/" + props.value + "/stages");
       setLoaded(true);
     }
-  }, [stages, isOpen]);
+  }, [stages, isOpen, loaded, props.value]);
 
   return (
     <Flex>