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

[YSQL] Replace copyright string YugaByteDB with YugabyteDB #23940

Closed
1 task done
pao214 opened this issue Sep 16, 2024 · 1 comment
Closed
1 task done

[YSQL] Replace copyright string YugaByteDB with YugabyteDB #23940

pao214 opened this issue Sep 16, 2024 · 1 comment
Assignees
Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue

Comments

@pao214
Copy link
Contributor

pao214 commented Sep 16, 2024

Jira Link: DB-12838

Description

Linter gives up with an internal offset error.

arc lint                                                       
 Exception 
Undefined offset: -1
(Run with `--trace` for a full exception trace.)

Issue Type

kind/bug

Warning: Please confirm that this issue does not contain any sensitive information

  • I confirm this issue does not contain any sensitive information.
@pao214 pao214 added area/ysql Yugabyte SQL (YSQL) status/awaiting-triage Issue awaiting triage labels Sep 16, 2024
@yugabyte-ci yugabyte-ci added kind/bug This issue is a bug priority/medium Medium priority issue labels Sep 16, 2024
@pao214 pao214 changed the title [YSQL] Fix linter error when the copyright string is wrong [YSQL] Replace copyright string YugaByteDB with YugabyteDB Sep 16, 2024
pao214 added a commit that referenced this issue Sep 16, 2024
Summary:
When running arc lint on a file that has

```
// Copyright (c) YugaByteDB, Inc.
```

the linter fails with an error

```
arc lint
 Exception
Undefined offset: -1
(Run with `--trace` for a full exception trace.)
```

The error message is not informative since this is an array index out of bounds exception within the linter. This makes it difficult for devs to root cause the issue.

This revision replaces the copyright string from the old version `YugaByteDB` to the correct version `YugabyteDB`.
Jira: DB-12838

Test Plan: Jenkins

Reviewers: amartsinchyk

Reviewed By: amartsinchyk

Subscribers: ybase, yql

Differential Revision: https://phorge.dev.yugabyte.com/D38073
jasonyb pushed a commit that referenced this issue Sep 17, 2024
Summary:
 f97d7d5 [#23770] [#23797] [#23837] YSQL: Fix most non-regress tests when run with Connection Manager
 ee2b108 [docs] reverted PR 23909 changes (#23941)
 bd80f4e [#23924] DocDB: Address recent flakiness of PgGetLockStatusTest.TestGetWaitStart
 Excluded: f0a5db7 [#20908] YSQL: Introduce interface to optimize index non-key column updates
 6556498 [#23897] xClusterDDLRepl: Support create table with partition by primary key
 e2b1d28 [#23363] Changing the default gflags of YSQL memory configuration.
 e717f43 [#23925] DocDB: Address recent regression of test PgWaitQueuesTest.MultiTabletFairness
 09b7702 [#23940] YSQL: Replace copyright string from YugaByteDB to YugabyteDB
 add83ef [#23947] Update callhome URL to use https
 69db717 Update faq page (#23704)
 Excluded: 063dbe5 [#23786] YSQL: yb_make_next_ddl_statement_nonincrementing
 Excluded: a913524 [#23859] YSQL: Remove redundant Bitmap Scan filters on partial indexes
 41f5afd [PLAT-15097]: Delete System platform DB table while disabling ysql
 d6bbf59 [#23890] docdb: Add filtering for bootstrap intent iterators based on min_replay_txn_start_ht
 0b37479 [#23770] YSQL: Stabalize TestPgExplainAnalyze#testExplainAnalyzeOptions the test with ysql connection manager

Test Plan: Jenkins: rebase: pg15-cherrypicks

Reviewers: jason, tfoucher

Subscribers: yql

Differential Revision: https://phorge.dev.yugabyte.com/D38123
@pao214 pao214 self-assigned this Sep 19, 2024
@pao214
Copy link
Contributor Author

pao214 commented Sep 19, 2024

Landed on master.

@pao214 pao214 closed this as completed Sep 19, 2024
@yugabyte-ci yugabyte-ci removed the status/awaiting-triage Issue awaiting triage label Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ysql Yugabyte SQL (YSQL) kind/bug This issue is a bug priority/medium Medium priority issue
Projects
None yet
Development

No branches or pull requests

2 participants