Frank Chen

Results 103 comments of Frank Chen

@kfaraz I added an override version to accept `Object... args` parameters to simplify. And also I added some comments to these functions to give suggestions that when 400 or 404...

CI failed because of insufficient line coverage.

@jihoonson Could you review this PR when you're at convenience? I saw there were conflicts with the latest changes on master, and worried about more conflicts.

@jihoonson A good suggestion. Let me create a new PR to fix the vulnerability only and leave this PR to unify the response model.

Description has been updated to reflect the latest change. And the old API will be added to the forbidden API list in later commits.

@kfaraz @jihoonson Could you please take a look at the changes in this PR when you're at convenience?

@xvrl I see the router and the coordinator are using jetty as http client. Do you know why we need two http client libraries? Is there any special consideration?

One tricky problem left is that first/last aggregators is not supported by SQL query on reindexed long/float/double columns while these aggregators work well in a native query. The type of...

Hi @clintropolis , Thanks for your suggestion. I'll try to solve it.

> > Hmm, so what I have had in mind to solve this is to be able to determine whether a `RowSignature` should be "finalized" or not in terms of...